- 19 Mar, 2021 10 commits
-
-
Benedikt Meurer authored
The `Script::source_url` field holds the value of the magic `//# sourceURL` comment if found, and the `Script::name` field is supposed to hold the actual name of the resource (as provided by the embedder ideally), in case of Chromium that's supposed to be the URL (in case of Node.js it's often the local path). Using `source_url` worked by chance so far, but for loading DWARF symbol files correctly we need the initiator (which we pick from the embedderName of the Script as reported to DevTools). More importantly, the partial handling of `//# sourceURL` in V8 is a layering violation and causes trouble in DevTools, i.e. when users put relative paths here. So as part of refactoring and correctifying the handling of `//# sourceURL`, we need to make sure that the embedder provided name (the URL in case of Chromium) is always stored in the `Script::name` field. Bug: chromium:1183990, chromium:974543, chromium:1174507 Change-Id: I32e11def2b9b52be11bd2e0e64a2ab6bdcf5e52d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773584 Commit-Queue: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by: Yang Guo <yangguo@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#73536}
-
Georgia Kouveli authored
This use of LR previously allowed overwriting it with arbitrary addresses that aren't signed. Change this so we never return to an arbitrary LR. Instead of loading the InterpreterTrampolineEntry address into LR directly, use an ADR instruction to place into LR the address of a piece of code that jumps to the InterpreterTrampolineEntry instead. This makes a difference because BR is also constrained by BTI, whereas RET isn't. An alternative would have been to `Call` instead of `Jump` to the target bytecode and avoid the ADR instruction altogether, but I wanted to keep the same behaviour with respect to the return stack that the existing code exhibits. Also add a comment to src/regexp/arm64/regexp-macro-assembler-arm64.cc for a similar use of LR that should eventually be removed. Bug: v8:10026 Change-Id: I24a13481f3fa416247dab8f9e5ae6f52f6b2ad42 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2764761Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Commit-Queue: Georgia Kouveli <georgia.kouveli@arm.com> Cr-Commit-Position: refs/heads/master@{#73535}
-
Shu-yu Guo authored
Calls with a spread expression in a non-final position get transformed to calls to Reflect.apply. This transformation is currently done in the parser, which does not compose well with other features (e.g. direct eval checking, optional chaining). Do this transform in the BytecodeGenerator instead. Bug: v8:11573, v8:11558, v8:5690 Change-Id: I56c90a2036fe5b43e0897c57766f666bf72bc3a8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2765783 Auto-Submit: Shu-yu Guo <syg@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73534}
-
Jakob Gruber authored
When making inlining decisions in the JSInliningHeuristic, it's possible that a Node is not a candidate on the first visit, but becomes a candidate in later visits due to other node reductions. These later visits should also result in the inlining decision being made. Until now this was prevented by the visit aborting early since the Node was added to the seen_ list on the first (unsuccessful) visit. This CL changes the seen_ insertion to happen only once a positive inlining decision was made. Change-Id: Ide7f6abd3c1d9759d7422fcd5ad9c7daff825795 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2764759 Auto-Submit: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Mythri Alle <mythria@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Reviewed-by: Mythri Alle <mythria@chromium.org> Cr-Commit-Position: refs/heads/master@{#73533}
-
Santiago Aboy Solanes authored
`has_non_instance_prototype` can be modified in live objects. For the native context's map that we serialize on the background this bit is "set" but it doesn't change value (i.e. it is set to false when it was already false). Bug: v8:7790, v8:11575 Change-Id: I070c0f0e383250d0e3cb92065d1113662976cabf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772609 Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#73532}
-
Santiago Aboy Solanes authored
Restore of https://chromium-review.googlesource.com/c/v8/v8/+/2194012. I changed it to be non-atomic and missed the fact that the concurrent marker accesses the has_prototype_slot concurrently. Bug: v8:7790, v8:11353 Change-Id: I292aeacb340b6340c40b633db2591c7d0cbca3bb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772608Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#73531}
-
Leszek Swirski authored
Bug: chromium:1011762 Change-Id: I960b3f57fa29f25d4cda038304825375ada686b9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2108039 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#73530}
-
Leszek Swirski authored
Add an always_sparkplug testing variant, and fix a couple of issues it found. Bug: v8:11420 Change-Id: I7d87a41e3413f40271a0140118531f075d633b23 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773047Reviewed-by: Patrick Thier <pthier@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73529}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/ff9d9de..100ae19 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/c93be42..4e07843 Rolling v8/third_party/aemu-linux-x64: fU_J9KtzLrfiXhqpf_1eV01gy30M641raluN5uaKPNAC..ASZAw9q3qc9gzTTRn-mGL72ir5Z_qIi5GvZGRBYa4sMC Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/2737963..9757ad5 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/41f9e2d..283bb29 Rolling v8/tools/luci-go: git_revision:e81c0c9c528d0a416922e4ccd958d0de59a64816..git_revision:ea8dc31395c76b2990112b29b02386628d795d2d Rolling v8/tools/luci-go: git_revision:e81c0c9c528d0a416922e4ccd958d0de59a64816..git_revision:ea8dc31395c76b2990112b29b02386628d795d2d Rolling v8/tools/luci-go: git_revision:e81c0c9c528d0a416922e4ccd958d0de59a64816..git_revision:ea8dc31395c76b2990112b29b02386628d795d2d TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Iefca39aa2842894aaa4ec039bad4862533cbf0e4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2774713Reviewed-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@{#73528}
-
Ng Zhi An authored
- Use movhlps instead of pshufd if dst == src - use xorps with punpckhdq instead of two shuffles Bug: v8:11470 Change-Id: I50f77b8591372cccca48b9a6078fc3132fe48a2f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773902Reviewed-by: Bill Budge <bbudge@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73527}
-
- 18 Mar, 2021 27 commits
-
-
Ng Zhi An authored
This is a reland of 1a69d8d8 The fix is to skip this regression test on systems that do not support SIMD. Original change's description: > [wasm-simd] Fix i32x4.extadd_pairwise_i16x8_u codegen > > It did not handle the case where dst == src. We switch the registers > used around to write to scratch first and ensure we don't overwrite dst. > > Bug: chromium:1187831 > Change-Id: Idf447aa1a3eff3920f2dfa3e0ec11efae37778cd > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2762425 > Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73521} Bug: chromium:1187831 Change-Id: I1bb6f4ce69f98064d3da5113e3b4c5a4f75a08c2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2774133Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73526}
-
Junliang Yan authored
Change-Id: I505ea67af31006f3f233eef390f234cfc0188d6c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2774479Reviewed-by: Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/master@{#73525}
-
Ng Zhi An authored
Move them from post-mvp to relaxed-simd flag. We will have a follow up change to move all the relaxed-simd tests into their own file. Bug: v8:11583 Change-Id: Iea9809a309bac428c856e5d0bd024fe0070d5921 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773898Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73524}
-
Junliang Yan authored
Change-Id: I1dd37b86634aae67a0cb7f0bf4d5b21e1f1845df Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773803Reviewed-by: Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/master@{#73523}
-
Clemens Backes authored
This reverts commit 1a69d8d8. Reason for revert: Fails on nosse: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux/41365/overview Original change's description: > [wasm-simd] Fix i32x4.extadd_pairwise_i16x8_u codegen > > It did not handle the case where dst == src. We switch the registers > used around to write to scratch first and ensure we don't overwrite dst. > > Bug: chromium:1187831 > Change-Id: Idf447aa1a3eff3920f2dfa3e0ec11efae37778cd > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2762425 > Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73521} Bug: chromium:1187831 Change-Id: I40d360b16338ef52e7e4327d4166c194ee950827 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773802 Auto-Submit: Clemens Backes <clemensb@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@{#73522}
-
Ng Zhi An authored
It did not handle the case where dst == src. We switch the registers used around to write to scratch first and ensure we don't overwrite dst. Bug: chromium:1187831 Change-Id: Idf447aa1a3eff3920f2dfa3e0ec11efae37778cd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2762425Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73521}
-
Leszek Swirski authored
In the TestTypeOf handler, use the statically known value of the type-of literal flag to emit the specific type-of check for that type, rather than going to the general type-of builtin. These checks are very simple, so we can get away with hand-writing them in the baseline compiler. Bug: v8:11420 Change-Id: Id9f51042916abaea62f929a2f95cf2c64dc32ee3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772613 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Patrick Thier <pthier@chromium.org> Reviewed-by: Patrick Thier <pthier@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73520}
-
Ng Zhi An authored
This is not used anywhere yet, and will be later used to gate instructions like FMA (which is prototyped). Bug: v8:11583 Change-Id: I2eac60cde6998490fcf3aee11c618ab5478346ee Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2770605Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73519}
-
Michael Achenbach authored
This reverts commit f187d0a1. Reason for revert: https://crbug.com/v8/11587 Original change's description: > [test] Extend testing on Mac on arm64 > > Ensure more testing of --future for different test types on Mac. > > No-Try: true > Bug: v8:11527 > Change-Id: Iac499dc48dde3342ad2057f86ef1ad5fa43b4eac > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772981 > Reviewed-by: Igor Sheludko <ishell@chromium.org> > Commit-Queue: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73514} Bug: v8:11527, v8:11587 Change-Id: I76a1baccdf20ed32b02fe8509ae13f7b62651592 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773051 Auto-Submit: Michael Achenbach <machenbach@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@{#73518}
-
Clemens Backes authored
The test was extended in this CL: https://crrev.com/c/2762420 It now uses wasm::SimdShuffle, which is only available if webassembly is enabled. Thus, #if out the test if webassembly is disabled. Drive-by: Add a missing include. R=jkummerow@chromium.org CC=zhin@chromium.org Bug: v8:11238 Change-Id: I1b53d0145467b58616a161944fb88d2ca256fd58 Cq-Include-Trybots: luci.v8.try:v8_linux64_no_wasm_compile_rel Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772978Reviewed-by: Zhi An Ng <zhin@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#73517}
-
Frank Emrich authored
This CL is part of a series that adds the C++ implementation of SwissNameDictionary, a deterministic property backing store based on Swiss Tables. This CL adds test-swiss-name-dictionary-infra.[h|cc], which contain the infrastructure for writing tests that simulatenously check the C++ and CSA/Torque implementation of SwissNameDictionary operations. The actual tests are added in a subsequent CL, which will be the last of this series. Bug: v8:11388 Change-Id: I89cbc7e575ed694fe34cb66c0e1ec70683504bd8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2742574Reviewed-by: Igor Sheludko <ishell@chromium.org> Reviewed-by: Marja Hölttä <marja@chromium.org> Commit-Queue: Frank Emrich <emrich@google.com> Cr-Commit-Position: refs/heads/master@{#73516}
-
Leszek Swirski authored
This reverts commit c4b44d5d. Reason for revert: UBSan failures (https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20UBSan/15621/overview) Original change's description: > [bigint] Begin src/bigint refactoring > > This patch moves a first function, Compare, from src/objects/bigint.cc > to src/bigint/, to blaze the trail. More to follow! > > Bug: v8:11515 > Change-Id: Id7fa0b40ea852dbed1360f7ab439cb32d0c15762 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2737295 > Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> > Reviewed-by: Hannes Payer <hpayer@chromium.org> > Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73511} Bug: v8:11515 Change-Id: I98f3e385c785297b1b71de07fc4835da2bbe816c No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773048 Auto-Submit: Leszek Swirski <leszeks@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@{#73515}
-
Michael Achenbach authored
Ensure more testing of --future for different test types on Mac. No-Try: true Bug: v8:11527 Change-Id: Iac499dc48dde3342ad2057f86ef1ad5fa43b4eac Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772981Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#73514}
-
Clemens Backes authored
We do not emit this opcode in the asm.js to wasm translator, hence we do not need to handle this case in the function body decoder. Replace it by a DCHECK to catch bugs where we might accidentally emit the opcode. R=thibaudm@chromium.org Change-Id: Ibaa07099f30b871c28e3d3cebae96277f46e3164 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2768619Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#73513}
-
Milad Fa authored
Port d16eefe0 Original Commit Message: We currently canonicalize shuffles in the architecture specific instruction selector. This has the drawback that if we want to pattern match on nodes that have a shuffle as input, they need to individually canonicalize the shuffle. There can also be a subtle bug if we canonicalize the same shuffle node twice (see bug for details). This moves the canonicalization to "construction time", in wasm-compiler, when building the graph. As such, any pattern matches in instruction-selector will only need to deal with canonicalized shuffles. We introduce a new kind of parameter for shuffle nodes, ShuffleParameter, to store the 16 bytes plus a bool indicating if this is a swizzle. A swizzle essentially: inputs to the shuffle are the same or all indices only touch 1 input. We calculate this when canonicalizing, so store this bit of information inside of the node's parameter. We update the tests in x64 to handle special cases where, even though the node's inputs are not swapped (due to canonicalization), they need to be swapped for the specific instruction selected (e.g. palignr). The test data also contains canonicalized shuffles, so we have to manually canonicalize them. R=zhin@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I9872fcdaa06739c8972f02d81e77bcbf372126c2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773138Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#73512}
-
Jakob Kummerow authored
This patch moves a first function, Compare, from src/objects/bigint.cc to src/bigint/, to blaze the trail. More to follow! Bug: v8:11515 Change-Id: Id7fa0b40ea852dbed1360f7ab439cb32d0c15762 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2737295Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> Reviewed-by: Hannes Payer <hpayer@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/master@{#73511}
-
Thibaud Michaud authored
Live ranges defined by a constant operand normally don't require a spill slot since they can just rematerialize the value from the constant. In the attached issue however, deoptimization adds an explicit slot requirement for a range that is defined by a constant operand. This case is not expected in the register allocator and we eventually hit a DCHECK. This fix allocates a new stack slot during the MeetRegisterConstraints and adds the missing gap move. Drive-by: remove dead method LiveRange::NextSlotPosition. R=sigurds@chromium.org CC=nicohartmann@chromium.org Bug: chromium:1146880 Change-Id: I08fbb890f2f3d9574196989cf3e5ef6232433484 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2563689Reviewed-by: Sigurd Schneider <sigurds@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#73510}
-
Ross McIlroy authored
Reduces the overhead of mid-tier optimization without much impact on mid-tier generated code performance. BUG=v8:9684 Change-Id: I81889049f718ec2b18a805b11aab119754466c95 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772611 Auto-Submit: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#73509}
-
Ross McIlroy authored
RepresentationFor is quite hot in MidTierAllocator profiles. To optimize this, instead stash the representation in the VirtualRegisterData and pass that about consistently instead of passing the virtual_register int and having to retrieve both representation and VirtualRegisterData for the vreg multiple times. This improves mid-tier allocation time by ~8% on Octane benchmarks. BUG=v8:9684 Change-Id: Ied01fbdab013c278da022d1df321b08fbfc68a4c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2768618Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#73508}
-
Georgia Kouveli authored
The previous uses of LR here allowed overwriting it with arbitrary addresses that aren't signed. Change this so we never return to an arbitrary LR. This makes a difference even when we replace the RET with a BR, because BR is constrained by BTI, whereas RET isn't. Bug: v8:10026 Change-Id: Ibbf326ccf0cf32f6d9541c7a82108dc0373827df Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2767015Reviewed-by: Georg Neis <neis@chromium.org> Commit-Queue: Georgia Kouveli <georgia.kouveli@arm.com> Cr-Commit-Position: refs/heads/master@{#73507}
-
Georgia Kouveli authored
Bug: v8:10026 Change-Id: Ie040d36d9a98943f97236998df53408bb060d028 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2764760Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Georgia Kouveli <georgia.kouveli@arm.com> Cr-Commit-Position: refs/heads/master@{#73506}
-
Igor Sheludko authored
... when v8_enable_short_builtin_calls is enabled for current build configuration. Bug: v8:11527 Change-Id: Ie7edf7ede5822f3b8f7ec815c89d0c50ec7657bc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2770880 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73505}
-
Thibaud Michaud authored
Fix target stack height of multi-value try blocks. R=clemensb@chromium.org Bug: chromium:1187896 Change-Id: I698b06141e65f7b545a695c035b862af31dd8875 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772236Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#73504}
-
Igor Sheludko authored
... because serialization of RUNTIME_ENTRY reloc infos is not supported yet. Bug: v8:11527 Change-Id: I0a0451710f7dd57b6393b8fad1620d29346a9f49 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772602Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#73503}
-
Manos Koukoutos authored
This reverts commit a3b1233e. Reason for revert: This approach has multiple issues and we have to reconsider it. Original change's description: > [turbofan] Optimize TrapIf/Unless in BranchElim. and CommonOp-Reducer > > Bug: v8:11510 > Change-Id: I1e8fcb54444e494c7d765ad556d09d954441361f > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2752876 > Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> > Reviewed-by: Georg Neis <neis@chromium.org> > Reviewed-by: Andreas Haas <ahaas@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73468} Bug: v8:11510 Change-Id: Id35bc4ebcb45a617f61993d857ad2291b0287ad6 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772600 Auto-Submit: Manos Koukoutos <manoskouk@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@{#73502}
-
Igor Sheludko authored
This is a reland of 5f9f09f4 Original change's description: > [sparkplug] Enable shorter builtin calls behind --short-builtin-calls > > ... on desktop x64 and arm64 if pointer compression is enabled. > > Bug: v8:11527 > Change-Id: Ie23b59312c6db34a5f40e23347b3c4f11173612d > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2767222 > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73498} Bug: v8:11527 Change-Id: I5b5524b6b4b301e4afcc09e4621942b77b56c07d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2772235Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#73501}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/37b76d6..ff9d9de Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/b965b65..c730dae Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/36de4be..2737963 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/752611e..41f9e2d Rolling v8/tools/luci-go: git_revision:19175e196dd495f6e092845f7d777cb5bf157b3e..git_revision:e81c0c9c528d0a416922e4ccd958d0de59a64816 Rolling v8/tools/luci-go: git_revision:19175e196dd495f6e092845f7d777cb5bf157b3e..git_revision:e81c0c9c528d0a416922e4ccd958d0de59a64816 Rolling v8/tools/luci-go: git_revision:19175e196dd495f6e092845f7d777cb5bf157b3e..git_revision:e81c0c9c528d0a416922e4ccd958d0de59a64816 TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I7a3f21d3d171596b6d98a843379bd1c146f2ae54 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2771598Reviewed-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@{#73500}
-
- 17 Mar, 2021 3 commits
-
-
Zhi An Ng authored
This reverts commit 5f9f09f4. Reason for revert: Broke https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Clusterfuzz%20Linux64%20-%20release%20builder/21079/overview Original change's description: > [sparkplug] Enable shorter builtin calls behind --short-builtin-calls > > ... on desktop x64 and arm64 if pointer compression is enabled. > > Bug: v8:11527 > Change-Id: Ie23b59312c6db34a5f40e23347b3c4f11173612d > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2767222 > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73498} Bug: v8:11527 Change-Id: I48ec54a134a293e31d4d26ce3a82a27a714fde6a No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2770599 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@{#73499}
-
Igor Sheludko authored
... on desktop x64 and arm64 if pointer compression is enabled. Bug: v8:11527 Change-Id: Ie23b59312c6db34a5f40e23347b3c4f11173612d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2767222 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#73498}
-
Thibaud Michaud authored
Throwing an object that needs special property lookup currently traps when we catch it in wasm. We should just return undefined to let the caller know that this is not a wasm exception object. Drive-by: use the named {caught_tag} register consistently. R=clemensb@chromium.org Bug: chromium:1188825 Change-Id: I8ebd4db756ac7ba04208ab43c7349c28b813fc49 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2767519Reviewed-by: Clemens Backes <clemensb@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#73497}
-