- 27 Apr, 2021 35 commits
-
-
Maya Lekova authored
Bug: v8:11695 Change-Id: I3b558260a23b4fb7d16bc8f1e9c57b94bce03838 No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2854299Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Auto-Submit: Maya Lekova <mslekova@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#74228}
-
Leszek Swirski authored
Remove a couple of cases where builtins called by the baseline code needed a context parameter, either because the interface descriptor accidentally included the context, or by adding a trivial context load from the frame where appropriate. Bug: v8:11420 Change-Id: I3f34cd452f42dd876255375ae357feb245c78cdc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848464 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
Nico Hartmann <nicohartmann@chromium.org> Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#74227}
-
Jakob Gruber authored
FixedDoubleArrays are a special case: 1 The reads are 64-bit and unaligned, thus use memcpy underneath. 2 The compiler only reads FDArray values for (constant) boilerplate elements. 1) makes proper atomic reads tricky-to-impossible without a lock. Luckily, 2) means we know that the array values are immutable after initialization, thus we can simply do a non-atomic read from the compiler thread. Bug: v8:7790 Change-Id: I39698d867543ce2214a2148511c5d90ced6364b3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848410 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#74226}
-
Michael Lippautz authored
- Move ASAN and MSAN support to base/ - Eval arguments for builds without sanitizer to preserve same semantics - IWYU fixes Bug: chromium:1056170 Change-Id: I212becf205e03b155c188ffd13cf5629df6f2cb8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851887Reviewed-by:
Omer Katz <omerkatz@chromium.org> Reviewed-by:
Clemens Backes <clemensb@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#74225}
-
Maya Lekova authored
Bug: v8:11700 Change-Id: Ibed6e02fdf17cef267135620a2ceab735be2ee9f No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2853593 Auto-Submit: Maya Lekova <mslekova@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by:
-
Camillo Bruni authored
The previously added perf-context Promise-hooks take a v8::Function as arguments. However, the builtin code was only accepting JSFunctions which causes cast errors. Drive-by-fix: Directly pass nativeContext in more places. Bug: chromium:1201465 Change-Id: Ic8bed11253a1f18a84e71eb9ea809b1ec1c3f428 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2850162Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#74223}
-
Michael Achenbach authored
The config was mistakenly switched to Bionic already. We keep it unspecified now to prepare reimaging the bots. No-Try: true Bug: chromium:1199430 Change-Id: Ib09d9abbe890fc891af69b6630bf21be85328196 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2853586 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Liviu Rau <liviurau@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Liviu Rau <liviurau@chromium.org> Cr-Commit-Position: refs/heads/master@{#74222}
-
Jakob Gruber authored
This is another step towards improving TSAN and test coverage of concurrent paths. By purging reconstructible (background-serialized or never-serialized) ObjectData instances from the cache after serialization, we increase ObjectData-construction activity on the background thread. Note that this means ObjectData is no longer canonicalized - two instances can point at the same underlying object. Losing this property is unfortunate, but we can revert back to it once work on the concurrency project is further advanced. Bug: v8:7790 Change-Id: I44b1366f61dc9087cddc76939512abed17e28d61 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2844661 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Georg Neis <neis@chromium.org> Auto-Submit: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#74221}
-
Maya Lekova authored
This CL makes the object passed as argument to IsLeafTemplateForApiObject be received as a handle instead of a raw C++ pointer. From the codegen point of view, the memory representation is the same, so this doesn't change its semantics. Bug: chromium:1052746 Change-Id: Ibc116aa4d577ba95f30d1014f15f34ef3fbb1a35 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851884Reviewed-by:
-
Wenyu Zhao authored
This CL ensures that the TPH object iterator is reset correctly before iteration. Bug: v8:11641 Change-Id: I041890f8a1999f5d0d75af017744a5618550f28c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849818Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#74219}
-
Patrick Thier authored
With --stress-scavenge it is possible that we don't have a memento for the object we are interested in anymore, making the test useless. Bug: v8:11701 Change-Id: I05984de33dc0c21d01023ebaa4ceebacfb63cc84 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851889 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Auto-Submit: Patrick Thier <pthier@chromium.org> Reviewed-by:
Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/master@{#74218}
-
Dan Elphick authored
Expose third_party/google_benchmark:google_benchmark as a public dep from third_party/google_benchmark:benchmark_main and a v8_libbase dependency to empty_benchmark to expose base/macros.h. Bug: v8:7330 Change-Id: I5205af365da2ac7c35647acff90ed65d831cfa07 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851888 Commit-Queue: Dan Elphick <delphick@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Auto-Submit: Dan Elphick <delphick@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#74217}
-
Wenyu Zhao authored
Bug: v8:11641 Change-Id: I2b7fb85fa4e2890787981af1559f535b76ec8b05 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849815Reviewed-by:
-
Maya Lekova authored
Bug: v8:11702 Change-Id: Ie6b03cce08b0c9c617ce58050ae4d066ba1f592a No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851891Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#74215}
-
Dan Elphick authored
Removes references to previously deleted headers: src/sanitizer/lsan-page-allocator.h src/base/atomicops_internals_portable.h src/base/atomicops_internals_std.h Bug: v8:7330 Change-Id: I8834d8041f50d4cd9402fc57490e8a6e0e7d5c76 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851885 Commit-Queue: Dan Elphick <delphick@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Auto-Submit: Dan Elphick <delphick@chromium.org> Reviewed-by:
-
Wenyu Zhao authored
TPH doesn't support multiple mutators at the moment, so concurrent_inlining is disabled. Thus we don't need to support `PublishPendingAllocations` for now. Bug: v8:11641 Change-Id: I26f63f4093027878563c15de594a0a6bbef31c0a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849819 Commit-Queue: Wenyu Zhao <wenyu.zhao@anu.edu.au> Reviewed-by:
-
Wenyu Zhao authored
* Failed tests that also trigger GCs are not filtered out. They are expected to be fixed when TPH supports real garbage collection. Bug: v8:11641 Change-Id: I30b8bcf48d5e3f32439eeffb39d28ee45db2a21c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849822Reviewed-by:
-
Wenyu Zhao authored
Place DCHECKs in per-space allocation functions and concurrent allocation functions to assert TPH is not enabled. For per-space allocation function, they should be redirected to TPH. For concurrent allocations, TPH doesn't support it at the moment. Bug: v8:11641 Change-Id: Ib7292bc03132353b0555d9cbd5b49936f4a6de76 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849817 Commit-Queue: Wenyu Zhao <wenyu.zhao@anu.edu.au> Reviewed-by:
-
Wenyu Zhao authored
Bug: v8:11641 Change-Id: I76226e98a298e7868ec4fda75cb1bb75b8d42b7f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849820Reviewed-by:
-
Daniel Dromboski authored
Update the way urllib is imported. As of Python 3, the old `urllib2` was split into several smaller modules under `urllib`. This commit unifies the resulting imported names across Python 2 and Python 3, for forward/backward compatibility. Bug: v8:9871 Change-Id: I81310ea83536269ae0cdf1406fd69285928c9357 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848488Reviewed-by:
-
QiuJi authored
1. add saving for s9, s10, s11 2. remove saving for gp. 3. fix maintain of fp (which alias to s0). Reference of calling convention source: https://riscv.org/wp-content/uploads/2015/01/riscv-calling.pdf. Change-Id: I7898f52e55df3976dccd736b72249f4fe28dd359 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848476Reviewed-by:
-
Leszek Swirski authored
Change-Id: I983d3c0a14238aeaa017f083b5df30668f6670e1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848472 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#74207}
-
Wenyu Zhao authored
Bug: v8:11641 Change-Id: I82434a2134862a2de9b878c564141168c844132d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849821Reviewed-by:
-
Georg Neis authored
Bug: v8:11692 Change-Id: I89cdb3d7102123f63fc66106c35962d0578222c9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851878 Auto-Submit: Georg Neis <neis@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#74205}
-
Michael Achenbach authored
Bug: chromium:1199430 Change-Id: I7bdbb6ecdf08e71b13825401ae0523a82386c48b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851882 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Liviu Rau <liviurau@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/84e217e..6273d81 Rolling v8/third_party/aemu-linux-x64: GCNw2-mtXN7PnLi5hLQH5ab_ViULLYtqr5C1KX36CYQC..CMJyNq6nZ3RoBUYmon01iRJ7MYYfVl4ds1kgbGFcNRMC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/1727be6..00b0b51 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/90aee2a..c91f5f6 Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/f16d43c..23ef295 TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I755a92fbcb724c599fc0e848ad8edb4e7461d23b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2852896Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#74203}
-
Santiago Aboy Solanes authored
This is a reland as-is of https://chromium-review.googlesource.com/c/v8/v8/+/2831488, which got lost in the js-heap-broker.cc to heap-refs.cc transition. Bug: v8:7790 Change-Id: I9dd9e77a665e70feb90457a906a221926a3c0b94 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848411 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Auto-Submit: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by:
-
Georg Neis authored
Bug: chromium:1202924 Change-Id: I555fc44c52a3883010e1c643a41d470fcc683a6a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851880 Auto-Submit: Georg Neis <neis@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Reviewed-by:
-
Michael Lippautz authored
Keep idiomatic state loop but rely on USE() to avoid warning about unused variables. Bug: v8:11687 Change-Id: Icde295723c5d389d827280f70a65776866d4e7a9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2850645 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
Lu Yahan authored
When add/sub with immm in [-4096, -2049] || [2048, 4094], it can be split two addi/subi instr. Change-Id: I94b93763c33fa5ef31c5ec4d23cbc5580a93ed1e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848732 Commit-Queue: Yahan Lu <yahan@iscas.ac.cn> Reviewed-by:
Brice Dobry <brice.dobry@futurewei.com> Cr-Commit-Position: refs/heads/master@{#74199}
-
Lu Yahan authored
Change-Id: I8cb07269fa6a9175af94daa6375677927927a497 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848100 Commit-Queue: Yahan Lu <yahan@iscas.ac.cn> Reviewed-by:
-
Lu Yahan authored
xori rd, rs, 0x1 mean is that negating bit 0 of rs. So we can delte xori and invert the condition of the branch. Change-Id: I318b7a2def6ec5d848757f85623564922abfcdc5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2847673 Commit-Queue: Yahan Lu <yahan@iscas.ac.cn> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/0ee8b27..84e217e Rolling v8/third_party/aemu-linux-x64: E8miK3g03NZQFrVhyywlfhYSWXsq2SfF7vw2pdW-doYC..GCNw2-mtXN7PnLi5hLQH5ab_ViULLYtqr5C1KX36CYQC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/8bc6b08..1727be6 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/61bf6e8..90aee2a Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/058fb58..86da5ec Rolling v8/third_party/icu: https://chromium.googlesource.com/chromium/deps/icu/+log/d25bdc0..cd9f9a9 TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ia892f8b95d147532ed915d4908871c0d5f2f309c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2852466Reviewed-by:
-
Lu Yahan authored
Change-Id: I7cf47d9be50790f453bd2908b58aff3a41e2f95b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2848735Reviewed-by:
-
Michael Lippautz authored
Bug: chromium:1056170 Change-Id: I0206078a672cb66edf6590430b35b7c3bc9ce1eb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2852238 Auto-Submit: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Reviewed-by:
-
- 26 Apr, 2021 5 commits
-
-
Zhi An Ng authored
This reverts commit d2ce5744. Reason for revert: We reverted the early canonicalization change, so we need to worry about non-canonicalized shuffles now. Original change's description: > [wasm-simd][arm64] Update f32x4.mul(dup) pattern matching > > We now canonicalize earlier in the pipeline, and don't need to worry > about non-canonicalized shuffles. > > Bug: v8:11542,v8:11257 > Change-Id: If9f5c44061465be339c98e479fd8c5a437bbd74b > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2778673 > Reviewed-by: Bill Budge <bbudge@chromium.org> > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73645} Bug: v8:11542 Bug: v8:11257 Change-Id: Ib492b3ab7ad140193975d2641999c12c9697e27b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2850630Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#74193}
-
Michael Lippautz authored
- Move LsanPageAllocator to base; - Use LsanPageAllocator in PageBackend that serves managed C++ objects; - Remove spurious TODO for GCInfoTable which should not use the LSAN-aware backend; Bug: chromium:1056170 Change-Id: I2caa11443ab44da5164f1c29339e302bffb49228 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2850157 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/665fcc3..0ee8b27 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/d0f3388..f6a8e55 Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/08f35c8..5844fc6 Rolling v8/third_party/aemu-linux-x64: RHTOD0RSgoWm-M1jtnmPhZKKrWS0SGcMPzXuBTCbIUYC..E8miK3g03NZQFrVhyywlfhYSWXsq2SfF7vw2pdW-doYC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/d628425..8bc6b08 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/efd29f6..61bf6e8 Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/7f27afe..058fb58 Rolling v8/third_party/icu: https://chromium.googlesource.com/chromium/deps/icu/+log/7e7574b..d25bdc0 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/d7dd881..ba668f1 Rolling v8/tools/luci-go: git_revision:0f913477afc80d5c4b6609834d3bef6b44910e67..git_revision:173195137e006c3bbbd2394409e3d752f8b3c62f Rolling v8/tools/luci-go: git_revision:0f913477afc80d5c4b6609834d3bef6b44910e67..git_revision:173195137e006c3bbbd2394409e3d752f8b3c62f Rolling v8/tools/luci-go: git_revision:0f913477afc80d5c4b6609834d3bef6b44910e67..git_revision:173195137e006c3bbbd2394409e3d752f8b3c62f TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I5e981c75993c85188e579264cc46f1c77c200b57 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2849981 Commit-Queue: Maya Lekova <mslekova@chromium.org> Reviewed-by:
-
Shu-yu Guo authored
https://chromium.googlesource.com/external/github.com/tc39/test262/+log/311265..70bc32 Bug: v8:7834 Change-Id: Ie2de0088d9baeaa2635749035030a7d86eee368d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2846157Reviewed-by:
Marja Hölttä <marja@chromium.org> Commit-Queue: Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/master@{#74190}
-
Maya Lekova authored
This CL ensures googletest is built with the build configuration used for other components of V8. This works around the issue that googletest is compiled with hidden visibility, even in configurations that compile with default visibility, such as when v8_enable_backtrace is provided. Bug: chromium:1191946 Change-Id: I70fa3ce0a668a71a091607c22d2dda67e496fec4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2850700Reviewed-by:
-
