- 31 Mar, 2021 7 commits
-
-
Maya Lekova authored
Skip debugger/wasm-inspect-many-registers which started failing recently on TSAN, until it's investigated further. Bug: v8:11610 Change-Id: Iaf2115d7cfbe82df279b3c4417c1ccd99a50b3b4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2796951 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Maya Lekova <mslekova@chromium.org> Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73738}
-
Leszek Swirski authored
Remove the requirement to pass the native context into the script cache, simple to be able to access the empty function. Instead, for script cache keys, use Smi::zero() in the 'owner function' slot. This allows CompileUnboundScript to be called outside of a Context scope. Change-Id: I9b4fe6dd43f14944728664f7203b748ced750e76 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794440 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#73737}
-
Dominik Inführ authored
MaybePrintAst needs to be run before parking the thread. It allows handle dereferences internally but that doesn't work when thread is parked. Bug: chromium:1193728 Change-Id: I0adbf6f236be8d442fd8be4d4d6e4931fcff4db6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794432Reviewed-by:
-
Peter Marshall authored
Bug: none Change-Id: Ifd381c70bc50ad85c7f2367c821be876b325d0fc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2793967 Auto-Submit: Peter Marshall <petermarshall@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
Yang Guo <yangguo@chromium.org> Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#73735}
-
Maya Lekova authored
This reverts commit c83c9590. Reason for revert: Speculatively reverting for a failure on Arm GC stress bot - https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Arm%20GC%20Stress/b8851256837192083520/overview Original change's description: > [ic] Add a new MegaDOM IC > > This patch implements the MegaDOM IC setup and access. A new MegaDOM > IC state indicates that we've seen only DOM accessors at this access > site. > > This CL only adds support for DOM getters in LoadIC, other kinds of > access will be added in follow on CLs. > > Still remaining TODO before shipping: > 1. Have a mechanism to invalidate the protector > 2. Have a mechanism to find the accessors that aren't overloaded > 3. Use a new builtin to miss to runtime on access check failure > > Change-Id: Ie12efe5e9fa284f023043b996d61e7d74e710ee2 > Bug: v8:11321 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2618239 > Reviewed-by: Omer Katz <omerkatz@chromium.org> > Reviewed-by: Camillo Bruni <cbruni@chromium.org> > Reviewed-by: Dan Elphick <delphick@chromium.org> > Reviewed-by: Mythri Alle <mythria@chromium.org> > Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73733} Bug: v8:11321 Change-Id: Ib6a55796f2a3c345d4923f9eaa215a6ff55ed15b No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794437 Auto-Submit: Maya Lekova <mslekova@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#73734}
-
Sathya Gunasekaran authored
This patch implements the MegaDOM IC setup and access. A new MegaDOM IC state indicates that we've seen only DOM accessors at this access site. This CL only adds support for DOM getters in LoadIC, other kinds of access will be added in follow on CLs. Still remaining TODO before shipping: 1. Have a mechanism to invalidate the protector 2. Have a mechanism to find the accessors that aren't overloaded 3. Use a new builtin to miss to runtime on access check failure Change-Id: Ie12efe5e9fa284f023043b996d61e7d74e710ee2 Bug: v8:11321 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2618239Reviewed-by:
Omer Katz <omerkatz@chromium.org> Reviewed-by:
Dan Elphick <delphick@chromium.org> Reviewed-by:
Mythri Alle <mythria@chromium.org> Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#73733}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/f14b4de..d01f8cf Rolling v8/buildtools/linux64: git_revision:b2e3d8622c1ce1bd853c7a11f62a739946669cdd..git_revision:5667cc61018864b17542e0baff8b790f245583b0 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/cbf9455..77b3c6b Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/cc80b4a..c9174a6 Rolling v8/third_party/aemu-linux-x64: XIdHv_hBF_KHdjhj04EhFab-lOzR9RJfY6u_7egTckgC..2zvSiyDoNIOA05-NfIzaaqrW2-5iI24Lkw3TJ8uGtFAC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/36e4502..1178627 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/9781115..f2cfd3d Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/303ff5a..ddfad9c Rolling v8/tools/luci-go: git_revision:e02b10c56fc68b83be5348df6a46cab27ca88fbc..git_revision:84c2e8c00bcf3bcea776a4b4a62134921d8a177a Rolling v8/tools/luci-go: git_revision:e02b10c56fc68b83be5348df6a46cab27ca88fbc..git_revision:84c2e8c00bcf3bcea776a4b4a62134921d8a177a Rolling v8/tools/luci-go: git_revision:e02b10c56fc68b83be5348df6a46cab27ca88fbc..git_revision:84c2e8c00bcf3bcea776a4b4a62134921d8a177a TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ia8bde8a54be27bc763755a0ce214b75a144e1800 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2796165Reviewed-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@{#73732}
-
- 30 Mar, 2021 15 commits
-
-
Adam Klein authored
Bug: v8:9337 Change-Id: Ic9b1072007ed19435b8ea669bee7c07c7eef8e2b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2795274 Auto-Submit: Adam Klein <adamk@chromium.org> Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#73731}
-
Junliang Yan authored
Change-Id: Ic1c0b725e810468938f2ac6b990cade8fd4204cc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794430Reviewed-by:
Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/master@{#73730}
-
Junliang Yan authored
Change-Id: I56728b6951940a8f0fad4934a015c798a823f578 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2795241Reviewed-by:
-
Zhi An Ng authored
This reverts commit 856e8577. Reason for revert: Broke https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux%20-%20debug/34562/overview Original change's description: > [dict-proto] SIMD support for SwissNameDictionary in Torque > > This CL adds a Torque-counterpart for swiss_table::GroupSse2Impl in > Torque. This allows the Torque version of SwissNameDictionary to use > SSE for lookups, rather than needing to bailout to the runtime on > x64/ia32. > > Bug: v8:11330 > Change-Id: I74e3f97c460a8b89031016967ec0e545265016a9 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787485 > Reviewed-by: Igor Sheludko <ishell@chromium.org> > Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> > Reviewed-by: Zhi An Ng <zhin@chromium.org> > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73727} Bug: v8:11330 Change-Id: I453b299455ac088f7e6ecde1e8003ef58d958b51 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2795270 Auto-Submit: Zhi An Ng <zhin@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#73728}
-
Igor Sheludko authored
This CL adds a Torque-counterpart for swiss_table::GroupSse2Impl in Torque. This allows the Torque version of SwissNameDictionary to use SSE for lookups, rather than needing to bailout to the runtime on x64/ia32. Bug: v8:11330 Change-Id: I74e3f97c460a8b89031016967ec0e545265016a9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787485Reviewed-by:
Igor Sheludko <ishell@chromium.org> Reviewed-by:
Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by:
Zhi An Ng <zhin@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#73727}
-
Milad Fa authored
Simd implementations on Power need P9 at a minimum. Feature is enabled on the simulator regardless of the host. Change-Id: I06bbfea11615457f5d837fe00c3dfe36b277d82b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787490Reviewed-by:
Junliang Yan <junyan@redhat.com> Reviewed-by:
-
Manos Koukoutos authored
Changes: - Wrap eqref and i31ref objects in the temporary wasm object wrapper (in addition to dataref and anyref). Accept those types in IsJSCompatibleSignature(). - Handle null correctly in all cases (i.e., do not wrap/unwrap it). - Improve some error messages. - Handle kRttWithDepth in one case where it was omitted. - Some small structure improvements. - Add an extensive test. Bug: v8:7748, v8:11606 Change-Id: Ie519f2c87421664dd02cf29fe94f9a9d7510bae2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794422 Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by:
Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#73725}
-
Milad Fa authored
Handler was changed on supported architectures in this CL: https://crrev.com/c/2786846 Change-Id: Ib7a63f12eccb956c4d7b84297f2f1514f053431a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787196Reviewed-by:
-
Jakob Gruber authored
Change-Id: I95a7438e25e68980023526de75b8e961b45d5301 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794423 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
-
Thibaud Michaud authored
Take locals into account when computing the stack index for the next control. Also include unwind in the list of blocks that have an implicit exception reference on the stack. R=ahaas@chromium.org Bug: chromium:1183774 Change-Id: I29c67d286f1ec5efa9f2f80e13d083d6eff5836e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794421Reviewed-by:
-
Leszek Swirski authored
Fix the InstallBaselineCode path in the InterpreterEntryTrampoline to restore the clobbered eax (i.e. argc) register. Bug: v8:11420, chromium:1192459 Change-Id: I97ce5739cf22a08fbb46dbf372ab6276bb802440 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791567 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/master@{#73721}
-
Thibaud Michaud authored
The delegate instruction is only allowed inside the try section of a try block. R=ahaas@chromium.org Bug: chromium:1193169 Change-Id: I71756b18903dc39e487721b401d18bc4e213cbff Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791565 Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Reviewed-by:
-
Thibaud Michaud authored
Before the introduction of exception handling in the compile fuzzer, stack overflows were deterministic because they always caused the whole stack to be unwound. With support for exception handling, the stack overflow exception can be caught and is a source of nondeterminism. Flag the interpreter result as such in this case so that we skip the correctness check. R=ahaas@chromium.org Bug: chromium:1193212 Change-Id: I7efd38df1b3c108c1c41997395c2b22ded2b4ad1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791566Reviewed-by:
-
Maya Lekova authored
The disabled regress-9017 test allocates too much memory and started flakily failing on TSAN. CL that triggered the flake: https://chromium-review.googlesource.com/c/v8/v8/+/2731528 Bug: v8:9337 Change-Id: Ia039e42a88c3665a9b9b4adc4a4b7c923afd5ba5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791564Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#73718}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/c835251..f14b4de Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/99a2527..5dbd89c Rolling v8/third_party/aemu-linux-x64: PQ5BA4WJfsqBSdlGRVmglasbA61vdJ1sHFa_R-euwHQC..XIdHv_hBF_KHdjhj04EhFab-lOzR9RJfY6u_7egTckgC Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/e0de6a8..9781115 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/fc39a84..303ff5a Rolling v8/tools/luci-go: git_revision:e1c81c53ccd0366e8fff438f89030043343d4d6b..git_revision:e02b10c56fc68b83be5348df6a46cab27ca88fbc Rolling v8/tools/luci-go: git_revision:e1c81c53ccd0366e8fff438f89030043343d4d6b..git_revision:e02b10c56fc68b83be5348df6a46cab27ca88fbc Rolling v8/tools/luci-go: git_revision:e1c81c53ccd0366e8fff438f89030043343d4d6b..git_revision:e02b10c56fc68b83be5348df6a46cab27ca88fbc TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I2902d385189a3278b6f6ac556f900ed649c585d4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2793149Reviewed-by:
-
- 29 Mar, 2021 18 commits
-
-
Ng Zhi An authored
Change-Id: I5a7619df5258665f015089986fbd679291d4f414 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791771Reviewed-by:
-
Ng Zhi An authored
Bug: v8:11589 Change-Id: Iaabea832006e68f9506c1e191d324cee46680e20 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791766Reviewed-by:
-
Milad Fa authored
Change-Id: Id9c82a83dca73aedd3ce98b648fc195aecb00c90 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791562Reviewed-by:
-
Milad Fa authored
After https://crrev.com/c/2773784, patch is also needed within test-run-wasm-relaxed-simd.cc. Change-Id: Id1efc905c52840eee7b382b90795574b889a7249 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2788408Reviewed-by:
-
Camillo Bruni authored
Bug: chromium:1061857 Change-Id: Ib87a18cae343c8861cf49e64923273a43db20960 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2767438Reviewed-by:
-
Camillo Bruni authored
the_hole_value is in ReadOnlyRoots and we don't need write barriers for it. Change-Id: Iebdac2e6a2c0f594445e80cfd016dda351a66f98 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787486Reviewed-by:
Sathya Gunasekaran <gsathya@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#73711}
-
Camillo Bruni authored
This Cl adds the two following flags to artificially slow down script execution in a controlled way: --script_run_delay delays the first every v8::Execute per isolate --script_run_delay_once delays every v8::Execute Bug: chromium:1193459 Change-Id: I78fcf940513e9f82fde57ff222e95df9202d00a7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2739641Reviewed-by:
-
Milad Fa authored
Change-Id: I50eef2e93a439a5b28a443ce6b9fce3296fc5ba1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2791561Reviewed-by:
-
Milad Fa authored
Change-Id: Ie79995eb81f2c69ad73757f74a5e4dbfb22b0481 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784003Reviewed-by:
Manos Koukoutos <manoskouk@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#73708}
-
Dominik Inführ authored
It could happen (e.g. with --stress-background-compile) that the main thread blocks for a background thread but the background thread requests a GC from the main thread. This would result in a deadlock. Avoid this by parking the main thread for potentially blocking operations and allow allocations while the main thread is parked. This CL introduces new states for the main thread: CollectionRequested and ParkedCollectionRequested. These states will force Safepoint(), Park() and Unpark() on the main thread into slow paths. The slow path can then perform a GC on the main thread - right before parking or after the main thread got unparked. Bug: v8:10315 Change-Id: If7ef31622d27320613139a0b7f79086fe3200f99 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2731528Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/master@{#73707}
-
Frank Emrich authored
The build time flag v8_dict_mode_prototypes is ill-named, because it does not control whether properties are kept in dictionary mode (this is done by the v8_dict_property_const_tracking flag), but instead it controls if SwissNameDictionary or NameDictionary is used as the property backing store for all dictionary mode objects. This CL renames the flag and updates its description. Change-Id: If1337838d1b6d8f089c281a77d9ef7cfd4007220 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786859Reviewed-by:
-
Marja Hölttä authored
Bug: v8:11525 Change-Id: I60566072f0af920fdf40ea5f1b80480910a224ce Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2778298 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
Manos Koukoutos authored
Bug: v8:7748, v8:11602 Change-Id: I007e80de0114cc651c8eb2ec825fc130a4b386c9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2790175Reviewed-by:
-
Thibaud Michaud authored
Don't switch to the catch state if it is never reached. Also steal the catch state instead of splitting it since it cannot be used after a delegate instruction. R=ahaas@chromium.org Bug: chromium:1192313 Change-Id: I3967ac81e066d2146c8aa37b26a35a99ba88bdf6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787488Reviewed-by:
-
Patrick Thier authored
With the addition of deoptimizing to baseline, we mark the begin of every bytecode as a valid jump target in baseline code (Required for CFI on arm64). Therefore we can omit marking excpetion handler positions and binds at the beginning of the bytecode as valid jump targets now. Bug: v8:11420 Change-Id: Id173dacb5534b680c5c3796c78e2a2c2288e5e0a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786841 Auto-Submit: Patrick Thier <pthier@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
-
Niek van der Maas authored
Bug: chromium:1192941 Change-Id: Ifb17736936df6f3b1ad6081e74d802be8897fe74 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787489Reviewed-by:
-
LiuYu authored
Port: 950b281f Change-Id: I77f9389d1dfeaccccad533aa1d7bf41f913e6ae3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2790502Reviewed-by:
Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Auto-Submit: Liu yu <liuyu@loongson.cn> Cr-Commit-Position: refs/heads/master@{#73700}
-
Dominic Chen authored
In inspector-task.cc, the frontend and backend runners are explicitly joined before going out-of-scope. On POSIX platforms, calling pthread_join() on a thread that has already been joined has undefined behavior. For example, under the musl C runtime library, a successful call to pthread_join() will unmap the pthread_t thread information region, and calling pthread_join() again will result in SIGSEGV. R=clemensb@chromium.org, szuend@chromium.org Change-Id: Ifdf34ed190df4c722c135ef043a3df588973b984 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2785905Reviewed-by:
Simon Zünd <szuend@chromium.org> Reviewed-by:
Clemens Backes <clemensb@chromium.org> Commit-Queue: Simon Zünd <szuend@chromium.org> Cr-Commit-Position: refs/heads/master@{#73699}
-
