- 18 Jan, 2021 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/3ecdb5e..43dd249 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/f46e9e7..cf567b6 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I22b2eb5734c9578289d1700b1fae88f2c338d3e7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2635361Reviewed-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@{#72125}
-
- 16 Jan, 2021 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/670a905..3ecdb5e Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/235cfe4..c38b5ab Rolling v8/buildtools/linux64: git_revision:595e3be7c8381d4eeefce62a63ec12bae9ce5140..git_revision:d62642c920e6a0d1756316d225a90fd6faa9e21e Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/b2af2da..f46e9e7 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/091f5ac..dabd965 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ib0c53b119f960f4d8d41d7bd1b4355ea82b0b009 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632878Reviewed-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@{#72124}
-
- 15 Jan, 2021 18 commits
-
-
Santiago Aboy Solanes authored
This reverts commit 3a6f75ac. Reason for revert: performance regressions https://bugs.chromium.org/p/chromium/issues/detail?id=1163063 Original change's description: > [objects] Remove MakeExternal case for uncached internal strings > > Concurrently accessing internal external uncached strings is not > thread-safe. We are removing a case where we can make such a string > through MakeExternal. > > Bug: v8:7790 > Change-Id: I958062c15cf40ccc330600bb572de98620866e54 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2565511 > Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71573} TBR=leszeks@chromium.org,solanes@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:7790 Change-Id: I5dcc734869c3c921eacd89426309141127a85f47 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2633547Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#72123}
-
Santiago Aboy Solanes authored
This reverts commit b3d09001. Reason for revert: https://chromium-review.googlesource.com/c/v8/v8/+/2565511 has to be reverted, and this was a follow-up to that Original change's description: > [objects] Remove uncached internal external string type > > We shouldn't be creating those anymore since they are not thread-safe. > > Bug: v8:7790 > Change-Id: I4546d995fa32eb076c8dfe9d95301fad719c9e07 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2615347 > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72006} TBR=rmcilroy@chromium.org,leszeks@chromium.org,solanes@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:7790 Change-Id: I4eb1a6b8446fa602eeb5bf29fbf1fe57182cdbf3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2627605Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#72122}
-
Andreas Haas authored
R=thibaudm@chromium.org Bug: v8:7581 Change-Id: I717466f045473015c8d99d1e640492486d05a832 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2625886 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#72121}
-
Michael Lippautz authored
Context objects are allocated on the heap and thus should be Data objects. This allows handling them through tracing in the GC through the API. Bug: chromium:1013149 Change-Id: Id3a7bfd57fab19a5669062ccf61c2f8588faf0bb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2627307Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#72120}
-
Andreas Haas authored
There is an implication from --predictable => --single-threaded => --no-wasm-tier-up. This implication introduces conflicting flags for all tests that set --wasm-tier-up. This CL disables all these tests. R=machenbach@chromium.org Bug: v8:11319 Change-Id: I32452d74ae33fc0e081971c5c0ed840334f274d5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632598 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#72119}
-
Thibaud Michaud authored
Replace 0x16 with 0x18 for the delegate opcode, to avoid a conflict with the function reference proposal. See https://github.com/WebAssembly/exception-handling/issues/145 R=clemensb@chromium.org Bug: v8:8091 Change-Id: Ib012f8680dfece200973e18fdf6c82877f10d5de Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632604Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#72118}
-
Sathya Gunasekaran authored
I haven't done any test262 rolls in forever and won't be doing any for the foreseeable future, there's no need for me to be an OWNER. Change-Id: I51d5915f5704d0c15c0735c95cfbff9464a0e929 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632592 Auto-Submit: Sathya Gunasekaran <gsathya@chromium.org> Reviewed-by: Shu-yu Guo <syg@chromium.org> Commit-Queue: Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/master@{#72117}
-
Thibaud Michaud authored
Exception ref is not part of the latest exception handling proposal. Remove the corresponding value type, the type opcode and some obsolete tests. R=clemensb@chromium.org Bug: v8:8091 Change-Id: I5e068ba3426f4b56b90ef056193acdd8cc8fe7d7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632599Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#72116}
-
Bill Budge authored
This is a reland of 2f3cda58 GetFirstUnusedStackSlot is restored, and used for finding the padding slot location, if any. Original change's description: > [compiler] Rework calculation to start of return slots > > - Changes GetOffsetToReturns to take into account return slot padding > and argument padding. > - Changes GetStackParameterDelta to use GetOffsetToReturns for the SP > delta calculation. > - Removes GetFirstUnusedStackSlot. > > Change-Id: I13df72e86750c62798bae262f0560cf1d7f981db > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2593306 > Commit-Queue: Bill Budge <bbudge@chromium.org> > Reviewed-by: Andreas Haas <ahaas@chromium.org> > Reviewed-by: Georg Neis <neis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72078} Change-Id: I954d7e7487728dbd6a545a6c84efb8f8f65d894d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2626979Reviewed-by: Georg Neis <neis@chromium.org> Reviewed-by: Andreas Haas <ahaas@chromium.org> Commit-Queue: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#72115}
-
Santiago Aboy Solanes authored
This reverts commit 37c3e05e. Reason for revert: https://chromium-review.googlesource.com/c/v8/v8/+/2565511 has to be reverted, and this was a follow-up to that Original change's description: > [test] Update MakeExternal tests > > They weren't initializing the VM at the start of the test. Also updated > the test description. > > Bug: v8:7790 > Change-Id: I7b9df9e3aebb43fc526e16ec260aa071c0fdeb92 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2615019 > Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71978} TBR=leszeks@chromium.org,solanes@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:7790 Change-Id: I35bc0090c3beed4ef11a458f0e26c44b22f7b654 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2627606Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#72114}
-
Andreas Haas authored
Liftoff is not fully feature complete yet. To test that Liftoff can bailout to TurboFan also for debugging, this CL adds * an opcode that is only implemented in TurboFan * a flag that allows that opcode to be compiled with TurboFan * a bailout for this opcode to Liftoff. R=clemensb@chromium.org Bug: v8:7581 Change-Id: Ie4b4654d0d36ab937a7dfe9b1bb6a187b17615fb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629284 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#72113}
-
Alex Kodat authored
76217f57 fixed the profiler so it would only sample a thread if it had the Isolate lock. Unfortunately, this fix missed a timing window where a thread might have the Isolate lock but might not have restored the thread-specific data such as thread_local_top_ for the locked thread yet, so the sampler might end up using data from a different thread. This doesn't cause any seg faults or the like because the thread we *meant* to sample has the Isolate lock so the thread we're accidentally sampling can't mess with any Isolate data but we can still get incorrect sample data which can be especially obvious if the accidentally sampled thread is inside code that would never run on the thread we meant to sample. Fortunately, we can tell when all thread-specific data has been restored to the Isolate because thread_state_ in the PerIsolateThreadData for a thread is set to a non-null value until everything has been restored, at which point it gets set to null. So the fix adds a check after the test for the Isolate lock to check if thread_state_ is null for the thread we mean to sample. If so, we know all the data in the Isolate is good to go for sampling. Bug: v8:11316 Change-Id: I02d6361d8cbd6ec809ad8fb7ef07f5e9c94c7d1e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2628133Reviewed-by: Peter Marshall <petermarshall@chromium.org> Commit-Queue: Peter Marshall <petermarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#72112}
-
Andreas Haas authored
R=zhin@chromium.org Change-Id: Ibf94191615d0081f089065e59c78b685e68f5bae Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629507 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#72111}
-
Mythri A authored
This is a reland of e38cb757. This was reverted as a potential culprit for a wasm failure. The actual revert that fixed the bots is here: https://chromium-review.googlesource.com/c/v8/v8/+/2630736. This should be safe to reland. I verified locally that the test is failing with or without this change. Original change's description: > [turboprop] Enable tierup to TurboFan with FLAG_turboprop > > FLAG_turboprop was used to test the turboprop compiler without any > further tierup to TurboFan. This cl changes: > - FLAG_turboprop to also tier up to TurboFan. > - Introduces FLAG_turboprop_as_toptier to continue running the > configuration without tierup. > - Removes FLAG_turboprop_as_midtier which is same as FLAG_turboprop. > > Bug: v8:9684 > Change-Id: I487bda13d226434837770ecc43b3ced7c31ccf19 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2622214 > Commit-Queue: Mythri Alle <mythria@chromium.org> > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> > Reviewed-by: Jakob Gruber <jgruber@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72101} Bug: v8:9684 Change-Id: I8b61fd8e562190c3c7bf5a003273f2a058542dad Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632588 Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#72110}
-
Liu Yu authored
Implement i64x2 widen i32x4, and saturating rounding multiply high. Port: 9cb7e571 Port: 39f2e3de Change-Id: Ib88aca3e3f66eb4a12be4d2cf4e51ac930849cc0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632353Reviewed-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@{#72109}
-
Liu Yu authored
Port: 646bdbf8 Bug: v8:10972 Change-Id: I9b199dc75d0e759a768da55298af383ebeb30e90 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2632351 Auto-Submit: Liu yu <liuyu@loongson.cn> Reviewed-by: Zhi An Ng <zhin@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#72108}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/d64e599..670a905 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/178c01b..b2af2da Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/64d94de..091f5ac Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/b12d1c8..ba4ee03 Rolling v8/tools/luci-go: git_revision:16e6d735358b0166f06fd2e4daa0da4cff9918e9..git_revision:77944aa535e42e29faadf6cfa81aee252807d468 Rolling v8/tools/luci-go: git_revision:16e6d735358b0166f06fd2e4daa0da4cff9918e9..git_revision:77944aa535e42e29faadf6cfa81aee252807d468 Rolling v8/tools/luci-go: git_revision:16e6d735358b0166f06fd2e4daa0da4cff9918e9..git_revision:77944aa535e42e29faadf6cfa81aee252807d468 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ie4615747dd11f2ce6c969dfa7f4cf661198d4c5a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2631788Reviewed-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@{#72107}
-
Zhi An Ng authored
Prototype load lane instructions on ARM Liftoff. We had a helper function for load lane that was living in instruction-selector. Move it out to assembler-arm so we can reuse that in Liftoff. Bug: v8:10975 Change-Id: Ic6e15c23eb778fb94a882609be622d2ca1f61ddb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2614225 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#72106}
-
- 14 Jan, 2021 20 commits
-
-
Ben Noordhuis authored
Remove the ambient dependency on the currently entered isolate, let the embedder pass it in explicitly. Bug: v8:11287 Change-Id: I03690390a308a59e2c6ea5c6ae268780d836b717 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2608209Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by: Yang Guo <yangguo@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#72105}
-
Milad Fa authored
Port f9e76d6d Original Commit Message: - Modify InstructionSelectors to track both padding and multiple slot values to correctly adjust stack pointers when pushing arguments. Pass stack offset as an immediate operand. - Modify CodeGenerators to handle alignment padding. R=bbudge@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I639b6394f33e5e694882cb98c43b58edf5d93775 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629947Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#72104}
-
Francis McCabe authored
This reverts commit 7fc1351b. Reason for revert: creating new flakiness? See https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20isolates/13052 Original change's description: > [wasm] Unskip a test > > Debugging with multiple isolates involved is fixed now, thus unskip the > test. > > R=thibaudm@chromium.org > > Bug: v8:10359, v8:10099 > Change-Id: I0134fe9da21c86dd40a559dbd4cf8c998a70be20 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629512 > Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72099} TBR=clemensb@chromium.org,thibaudm@chromium.org Change-Id: Ie328a3af76ba2070cbd0971395d6414d48f4d973 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:10359 Bug: v8:10099 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2630736Reviewed-by: Francis McCabe <fgm@chromium.org> Commit-Queue: Francis McCabe <fgm@chromium.org> Cr-Commit-Position: refs/heads/master@{#72103}
-
Francis McCabe authored
This reverts commit e38cb757. Reason for revert: Test failing: https://logs.chromium.org/logs/v8/buildbucket/cr-buildbucket.appspot.com/8858103866497469056/+/steps/Check/0/logs/tier-down-to-liftoff/0 Original change's description: > [turboprop] Enable tierup to TurboFan with FLAG_turboprop > > FLAG_turboprop was used to test the turboprop compiler without any > further tierup to TurboFan. This cl changes: > - FLAG_turboprop to also tier up to TurboFan. > - Introduces FLAG_turboprop_as_toptier to continue running the > configuration without tierup. > - Removes FLAG_turboprop_as_midtier which is same as FLAG_turboprop. > > Bug: v8:9684 > Change-Id: I487bda13d226434837770ecc43b3ced7c31ccf19 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2622214 > Commit-Queue: Mythri Alle <mythria@chromium.org> > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> > Reviewed-by: Jakob Gruber <jgruber@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72101} TBR=rmcilroy@chromium.org,mythria@chromium.org,jgruber@chromium.org Change-Id: Ic3e87c311fba001460e4f1561a2e5f74391a06a7 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:9684 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2630526Reviewed-by: Francis McCabe <fgm@chromium.org> Commit-Queue: Francis McCabe <fgm@chromium.org> Cr-Commit-Position: refs/heads/master@{#72102}
-
Mythri A authored
FLAG_turboprop was used to test the turboprop compiler without any further tierup to TurboFan. This cl changes: - FLAG_turboprop to also tier up to TurboFan. - Introduces FLAG_turboprop_as_toptier to continue running the configuration without tierup. - Removes FLAG_turboprop_as_midtier which is same as FLAG_turboprop. Bug: v8:9684 Change-Id: I487bda13d226434837770ecc43b3ced7c31ccf19 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2622214 Commit-Queue: Mythri Alle <mythria@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#72101}
-
Santiago Aboy Solanes authored
This reverts commit 16e5dca3. Reason for revert: https://chromium-review.googlesource.com/c/v8/v8/+/2565511 has to be reverted, and this was a follow-up to that Original change's description: > [compiler] Mark String as kNeverSerialized > > Bug: v8:7790 > Change-Id: I2deb462c3467f7239d55b0f295feed1de5ca1c2f > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2616198 > Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> > Reviewed-by: Georg Neis <neis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72010} TBR=neis@chromium.org,solanes@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:7790 Change-Id: Ie8b628fc5d0b0d90637515d7d3a2f1a9f64a0a09 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629747Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#72100}
-
Clemens Backes authored
Debugging with multiple isolates involved is fixed now, thus unskip the test. R=thibaudm@chromium.org Bug: v8:10359, v8:10099 Change-Id: I0134fe9da21c86dd40a559dbd4cf8c998a70be20 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629512Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#72099}
-
Andreas Haas authored
R=clemensb@chromium.org Bug: v8:10994 Change-Id: I2145559f6b9f4d378a0b8c063133e8d59b7ce4d2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629510Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#72098}
-
Santiago Aboy Solanes authored
This reverts commit 73875e95. Reason for revert: https://chromium-review.googlesource.com/c/v8/v8/+/2565511 has to be reverted, and this was a follow-up to that Original change's description: > [compiler] Mark Name as kNeverSerialized > > Bug: v8:7790 > Change-Id: I44469e08131ad6a5f95a465cf2d461da0785221e > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2616218 > Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> > Reviewed-by: Georg Neis <neis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72012} TBR=neis@chromium.org,solanes@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:7790 Change-Id: I300c7050522de3c5f44cb4466e87f3388f221725 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2627603Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#72097}
-
Ulan Degenbaev authored
Bug: v8:11181 Change-Id: I8ca8b7249ef660874da761c11f192ffd06748ff5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2558219 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#72096}
-
Patrick Thier authored
Offsets in regular expressions are limited to 16 bits. It was possible to exceed this limit when emitting greedy loops where the length of text nodes exceeded 16 bits, resulting in overflowing offsets. With this CL we throw a SyntaxError "Regular expression too large" to prevent this overflow. Bug: chromium:1166138 Change-Id: Ica624a243bf9827083ff883d9a976f13c8da02e5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629286 Commit-Queue: Patrick Thier <pthier@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#72095}
-
Manos Koukoutos authored
Changes: - Add --wasm-loop-unrolling flag. Everything in this CL happens behind this flag. - In decoding, DoReturn does not take returned values as an argument. It is now the responsibility of graph-builder-interface.cc to extract these values. Note that this is what was already happening in Liftoff. - In pipeline.cc, add phase to remove loop exits after generating the turbofan graph. - Explicitly disallow calling FallThruTo() on loops. - Add loop assignments and loop header node to Control type in graph-builder-interface.cc. Assign them in Loop(). - Main change: Add loop exit nodes to wasm-generated graphs. For details, consult this design doc: https://docs.google.com/document/d/1AsUCqslMUB6fLdnGq0ZoPk2kn50jIJAWAL77lKXXP5g - Inline PrepareForLoop(). Bug: v8:11298 Change-Id: I65058f1b5df3f862f4a62f4dcb0bd7e1f1dcf4ee Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2621082 Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#72094}
-
Dan Elphick authored
This means that targets like pdfium don't have to manually set up the include dir. Bug: v8:11292 Change-Id: Idde914592a86f2f5a3fc5cf97060d5650467cc7a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629280 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@{#72093}
-
Liu Yu authored
Port: e14de8b9 Bug: v8:10971 Change-Id: Ia193010133ffc4ac49268ed095cdda8f7b732234 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629110 Auto-Submit: Liu yu <liuyu@loongson.cn> Reviewed-by: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Cr-Commit-Position: refs/heads/master@{#72092}
-
Lutz Vahl authored
TBR=vahl@chromium.org Change-Id: I4a1ebf06681f7618392b8d4915396e6edaf5dc10 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2629277Reviewed-by: Lutz Vahl <vahl@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Lutz Vahl <vahl@chromium.org> Auto-Submit: Lutz Vahl <vahl@chromium.org> Cr-Commit-Position: refs/heads/master@{#72091}
-
Michael Lippautz authored
Adds support for Member<const T> by keeping the untyped storage in MemberBase const, which is stronger than the required constness. All accesses go through BasicMember which can re-add the appropriate constness specified by the user. The same concept is applied to all Member and Persistent handles. Bug: chromium:1056170 Change-Id: I5a620258be3acb6a1b4b1437e69b8d7d1ec5ce6f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2625871Reviewed-by: Omer Katz <omerkatz@chromium.org> Reviewed-by: Anton Bikineev <bikineev@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#72090}
-
John Xu authored
The wrappers target currently only has one header. Marking it as source set created trouble on Windows and Mac builders. Bug: v8:11303 Change-Id: I228d49c4150eb089e8494555886072b237b95940 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2626855Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Auto-Submit: John Xu <johnx@google.com> Cr-Commit-Position: refs/heads/master@{#72089}
-
Jakob Gruber authored
This is a reland of 164cf80b The reland fixes UB (left-shift of negative integer type) with a static_cast<uint32_t>. Original change's description: > [regexp] Hard-crash on invalid offsets in AdvanceCurrentPosition > > Drive-by: Range checks in `Emit(byte, twenty_four_bits)` to ensure the > given packed bits actually fit into 24 bits. > > Bug: chromium:1166138 > Change-Id: I2e711e6466bb48d7b9897f68dfe621d12bd92508 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2625877 > Commit-Queue: Jakob Gruber <jgruber@chromium.org> > Commit-Queue: Leszek Swirski <leszeks@chromium.org> > Auto-Submit: Jakob Gruber <jgruber@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/master@{#72064} Tbr: leszeks@chromium.org Bug: chromium:1166138 Change-Id: I514495e14bb99dfc9588fdb4a9f35d67d8d64acb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2626663Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#72088}
-
Zhi An Ng authored
Prototype load lane instructions on ARM64 Liftoff. We extract a helper function that will convert a MemOperand that uses a register offset or a non-zero immediate offset, into a MemOperand that uses a temporary to store base+offset and immediate offset of 0. Some instructions, ld1r or ld1 don't support offset, and will use this helper to get the correct MemOperand. Bug: v8:10975 Change-Id: I89d133af7a706a49985db36ca044ac8956c6e1d5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2614171 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#72087}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/787a10d..d64e599 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/3889691..178c01b Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/8149a96..64d94de TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I9827b870666a957cf9414093662c1581ccbfcac5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2628620Reviewed-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@{#72086}
-