- 25 Aug, 2022 20 commits
-
-
Michael Achenbach authored
Bugs that are older than the switch of v8_foozzie.py to Python3 bisect to the switch commit unfortunately. This change attempts to let bisect run longer if a python2 executable still exists. No-Try: true Bug: chromium:1355824 Change-Id: I457a50af21704ddd2985793861eee8be5601a673 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3856574Reviewed-by: Liviu Rau <liviurau@google.com> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#82728}
-
Dominik Inführ authored
ThinStrings are not transitioned to other string representations, so we shouldn't need the snapshotting protocol for them. Change-Id: I17cee1a4171b10c441a005ac29bd232a0a065207 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3852489Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/main@{#82727}
-
Victor Gomes authored
This does not enable exception handlers yet, we still bail out in MaglevCompiler::Compile if we have an exception handler table in the bytecode array. This CL: - Generates code for exception handler blocks (which previously were set as dead code) - Creates a machinery for nodes to set the property CanThrow - Reads the exception handler table from the bytecode array and identifies if we're emitting nodes inside a try-block and for which handler we should jump in case of an exception - Generates an exception handler table for Maglev code Change-Id: Ifc9d4cb7440d3222f4fda48a86e4e482340b3b15 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854061 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#82726}
-
Feng Yu authored
Bug: v8:12781 Change-Id: I281047a0606b8f709a930998ef7e7a53f780f59d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3840146Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Feng Yu <f3n67u@gmail.com> Cr-Commit-Position: refs/heads/main@{#82725}
-
Feng Yu authored
This is a reland of commit 437b311a Original change's description: > [test] Migrate cctest/test-inspector to unittests/ > > test-inspector.cc -> inspector-unittest.cc > > Bug: v8:12781 > Change-Id: I37d2bc2d023ffd91b94d5a09cdbfe4a6e22fecf1 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3813062 > Reviewed-by: Jakob Linke <jgruber@chromium.org> > Commit-Queue: Jakob Linke <jgruber@chromium.org> > Reviewed-by: Hannes Payer <hpayer@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82512} Bug: v8:12781 Change-Id: Iece26e724f21d459dd1e96423d3aa72f15c5424e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3837705 Commit-Queue: Feng Yu <f3n67u@gmail.com> Reviewed-by: Jakob Linke <jgruber@chromium.org> Reviewed-by: Hannes Payer <hpayer@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#82724}
-
Victor Gomes authored
Bug: v8:7700 Change-Id: I1cc74a53cde8594b3f0b116c7876e0d70139084c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3856583Reviewed-by: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/main@{#82723}
-
Leszek Swirski authored
--stress-opt never did what we wanted it to; it ran its runs in different contexts (therefore not able to share feedback across runs), and even if it didn't, each run would create new closures for any defined closures, so we'd still more than likely end up poly- or mega-morphic. Fuzzers cover this use case better than --stress-opt ever did, so now it's just using precious bot time. We can get rid of it. Bug: v8:10386 Change-Id: Ibbb9207d887b4b1dc4ec9093858d477c0f95eb37 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3803228 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#82722}
-
Igor Sheludko authored
This reverts commit c89998d9. Reason for revert: a lot of unexpected memory regressions that require investigation. Original change's description: > Reland "[ext-code-space] Enable Code-less embedded builtins" > > This is a reland of commit 40901824 > The Ubsan issue is fixed here: https://chromium-review.googlesource.com/c/v8/v8/+/3849038. > > Original change's description: > > [ext-code-space] Enable Code-less embedded builtins > > > > Bug: v8:11880, v8:12592 > > Change-Id: I8d3d6ad0a4c26eb1fea2a998ffeddd1d96afa690 > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3784593 > > Commit-Queue: Igor Sheludko <ishell@chromium.org> > > Reviewed-by: Jakob Linke <jgruber@chromium.org> > > Cr-Commit-Position: refs/heads/main@{#82642} > > Bug: v8:11880, v8:12592 > Change-Id: I66373d6af30b060d1204b952d733e260228548df > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3846493 > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Reviewed-by: Jakob Linke <jgruber@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82683} Bug: v8:11880, v8:12592, chromium:1356329 Change-Id: I07aaf714da8d7afc66cf4116f189dd5dde5b8818 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3856357Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82721}
-
Al Muthanna Athamina authored
Bug: v8:13234 No-Try: true Change-Id: I63a5402ce5e4419972e0d6728c7615a341398648 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855450Reviewed-by: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Almothana Athamneh <almuthanna@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#82720}
-
Feng Yu authored
This is a reland of commit 3615ae69. Original change's description: > [test] Migrate cctest/test-global-handles to unittests/ > > Bug: v8:12781 > Change-Id: If7681564f3e0c087e3347557a3f9169625b51607 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3817621 > Reviewed-by: Camillo Bruni <cbruni@chromium.org> > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82633} Bug: v8:12781 Change-Id: I776ab176733722fd8853deaabec93e0d44274d9c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855979Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#82719}
-
ishell@chromium.org authored
This CL simplifies the API calls by removing some instructions from the most common path. Bug: v8:11880 Change-Id: Id8a62c35af51947ad2c152e093346d03c8e2f508 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855039 Auto-Submit: Igor Sheludko <ishell@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#82718}
-
Feng Yu authored
This is a reland of commit d83346be Original change's description: > [test] Migrate cctest/test-weakmaps to unittests/ > > Bug: v8:12781 > Change-Id: Ief6bd7ee0ff2876e19970b2fb6af4f3208ec7f4e > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3815486 > Commit-Queue: Leszek Swirski <leszeks@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82699} Bug: v8:12781 Change-Id: I9f74c32b924433dea67f62e26a336ef7263282e7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3856096 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#82717}
-
Tom Anderson authored
Bug: chromium:1355871 Change-Id: I7c465e7ce326c62cbc8ad92e2547eea78d23fe0b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3852443Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/main@{#82716}
-
Vasili Skurydzin authored
Change-Id: I99b4c5d9ea6183acf629f5f76ebb0f84c88e7b20 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855198 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Milad Farazmand <mfarazma@redhat.com> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#82715}
-
Clemens Backes authored
Add static asserts that we only use specific types for flag values. Also, document that string values are not be frozen yet, and add TODOs to fix that. R=cbruni@chromium.org Bug: v8:12887 Change-Id: I7367108810f0c6463509f744c5cefd9392c469fb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3852487Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#82714}
-
Leszek Swirski authored
This reverts commit d83346be. Reason for revert: Seems to be breaking TSAN bot (https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20stress-incremental-marking/8867/overview) Original change's description: > [test] Migrate cctest/test-weakmaps to unittests/ > > Bug: v8:12781 > Change-Id: Ief6bd7ee0ff2876e19970b2fb6af4f3208ec7f4e > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3815486 > Commit-Queue: Leszek Swirski <leszeks@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82699} Bug: v8:12781 Change-Id: Ia20078e9f40d0e2257cf0b953ddd2e72594c4085 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855938 Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82713}
-
Liu Yu authored
Port commit 805df68f Bug: v8:11708 Change-Id: I1a6970d18f1991e4158edb41ec5b8e393fccba4c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855696 Auto-Submit: Liu Yu <liuyu@loongson.cn> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Reviewed-by: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Cr-Commit-Position: refs/heads/main@{#82712}
-
Clemens Backes authored
This reverts commit d84b4664. Reason for revert: Fails "Mutable Constants" check on android-binary-size: https://ci.chromium.org/ui/p/chromium/builders/try/android-binary-size/1211670/overview For details about this check, see https://chromium.googlesource.com/chromium/src/+/main/docs/speed/binary_size/android_binary_size_trybot.md#Mutable-Constants Original change's description: > [flags] Rename v8_flags to FLAGS > > Team members expressed concerns that "v8_flags" is easier to miss in the > code than the previous "FLAG_" syntax. After a poll and discussions we > decided to rename the struct to "FLAGS", so the new syntax for > addressing flag values is "FLAGS.foo" instead of the previous > "FLAG_foo". > > R=cbruni@chromium.org > CC=jkummerow@chromium.org > > Bug: v8:12887 > Change-Id: I51af4aa7fd5a3b3c29310c0cb4c4ff42086ff012 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854508 > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Reviewed-by: Camillo Bruni <cbruni@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82701} Bug: v8:12887 Change-Id: I75516a0be9bc475afa2bbaa96a05e8a9b5be9be7 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855936 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#82711}
-
Lu Yahan authored
Bug: v8:11708 Change-Id: I803b5499f1bbc3f7b4e626628a73f98239df8454 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854435 Auto-Submit: Yahan Lu <yahan@iscas.ac.cn> Commit-Queue: ji qiu <qiuji@iscas.ac.cn> Reviewed-by: ji qiu <qiuji@iscas.ac.cn> Cr-Commit-Position: refs/heads/main@{#82710}
-
Michael Lippautz authored
Resident set size may be smaller than the recorded size in StatsCollector due to discarded memory. Change-Id: I7e052fc4412afc64dc1ed5be6ed7dc9271e6f9d2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855204 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Anton Bikineev <bikineev@chromium.org> Commit-Queue: Anton Bikineev <bikineev@chromium.org> Auto-Submit: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#82709}
-
- 24 Aug, 2022 20 commits
-
-
Tobias Tebbi authored
This improved Turboshaft value numbering performance significantly in local testing. Change-Id: I8b8c8e3a67da9b83b57efc34c08ca693e84d9076 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3852486Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Auto-Submit: Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/main@{#82708}
-
Deepti Gandluri authored
Change-Id: Ic462c0302b9dcb9cc269f99c091610235f2cce56 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3855037Reviewed-by: Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/main@{#82707}
-
Jakob Kummerow authored
Computation of this constant should obviously use kMaximumReprSizeLog2. It's unclear if this could have caused observable misbehavior. Change-Id: Iafdcbeb77d582f5f4e4aad07581377b74bb776c6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854316Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Auto-Submit: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/main@{#82706}
-
Jakob Kummerow authored
When walking the stack and visiting compressed spill slots, maintain their compressedness so that generated code can rely on spilled values not magically changing. Tested manually using the benchmark in the associated bug, as I'm not sure how to create a fast, reliable regression test for this. Fixed: v8:13216 Change-Id: Iebd1fb513975d9ee2567f7141f3ab18a04b0f4e1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854507 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#82705}
-
Jakob Kummerow authored
When we spill a register that we know contains only 32 interesting bits and then reload it from the spill slot, it's enough to reload its lower half. This may save a few bytes, and guards against accidental changes to the upper half (e.g. via pointer decompression). Bug: v8:13216 Change-Id: I1d950d6e33d8ae94cf385af4f3e1db028bf333c5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854506Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#82704}
-
Clemens Backes authored
Many tests have a long execution time already, and running them in stress mode is unlikely to flush out bugs (spec tests are supposed to check for spec-conform behaviour, and this is unlikely to change if run multiple times). R=jkummerow@chromium.org Bug: v8:13195 Change-Id: I029102e31f1e2e240e02376fbd5cd40ff0acc07a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3852488Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#82703}
-
Jakob Kummerow authored
On arm, SIMD registers alias with pairs of double registers. When deciding where to allocate the parameter values, we expect to see all register-passed parameters before all stack-passed parameters; but due to s128 and f64 params being arbitrarily interleaved this doesn't always hold. This patch fixes that by first finding all registers used for parameters, and then blocking these when allocating registers for other parameters. Fixed: chromium:1355070 Change-Id: I20deace58b960a9d1a5e3b794c46011f8f31b333 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854497Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#82702}
-
Clemens Backes authored
Team members expressed concerns that "v8_flags" is easier to miss in the code than the previous "FLAG_" syntax. After a poll and discussions we decided to rename the struct to "FLAGS", so the new syntax for addressing flag values is "FLAGS.foo" instead of the previous "FLAG_foo". R=cbruni@chromium.org CC=jkummerow@chromium.org Bug: v8:12887 Change-Id: I51af4aa7fd5a3b3c29310c0cb4c4ff42086ff012 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854508 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#82701}
-
Leszek Swirski authored
Since the function entry stack check happens outside of the IR, the standard register spilling mechanisms don't kick in and registers that expect to be valid might be clobbered. The only such case is, in fact, the new.target register, so make sure it is preserved across the stack check. R=jgruber@chromium.org Bug: v8:7700 Change-Id: I530b6af882ca188b0e3c7da752f810506f3340a0 Fixed: v8:13226, chromium:1356082 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3852389 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#82700}
-
Feng Yu authored
Bug: v8:12781 Change-Id: Ief6bd7ee0ff2876e19970b2fb6af4f3208ec7f4e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3815486 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#82699}
-
Samuel Groß authored
This reverts commit eca383c9. Reason for revert: Still seeing DCHECK failures in SweepAndCompact Original change's description: > Reland "[sandbox] Sandboxify EmbedderDataSlots" > > This is a reland of commit e1f585ed > > ExternalPointerTable issues have been fixed in > https://crrev.com/c/3849650 and https://crrev.com/c/3849376 > > Original change's description: > > [sandbox] Sandboxify EmbedderDataSlots > > > > Bug: v8:10391 > > Change-Id: If85a308a6f6ed1b17d86f87b4911c82d2327ea72 > > Cq-Include-Trybots: luci.v8.try:v8_linux64_heap_sandbox_dbg_ng,v8_linux_arm64_sim_heap_sandbox_dbg_ng > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3757341 > > Reviewed-by: Igor Sheludko <ishell@chromium.org> > > Commit-Queue: Samuel Groß <saelo@chromium.org> > > Cr-Commit-Position: refs/heads/main@{#82623} > > Bug: v8:10391 > Change-Id: If77f6c10e81c30c2dfa6b33c788bc4a36e4da135 > Cq-Include-Trybots: luci.v8.try:v8_linux64_heap_sandbox_dbg_ng,v8_linux_arm64_sim_heap_sandbox_dbg_ng > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3852602 > Commit-Queue: Samuel Groß <saelo@chromium.org> > Reviewed-by: Igor Sheludko <ishell@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82686} Bug: v8:10391 Change-Id: Icaa1ff64cabd1bb2f19d9b019eac0ca98e528eb6 Cq-Include-Trybots: luci.v8.try:v8_linux64_heap_sandbox_dbg_ng,v8_linux_arm64_sim_heap_sandbox_dbg_ng No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854677Reviewed-by: Leszek Swirski <leszeks@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#82698}
-
Camillo authored
Bug: chromium:1355059 Change-Id: I0e654660501ed56ad73d76faeb371733de38af2b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854505 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#82697}
-
George Wort authored
This reverts commit aa541f1c. Reason for revert: Reverting due to large regressions for motionmark on M1. Original change's description: > [turbofan][arm64] Emit Lsl for Int32MulWithOverflow when possible > > Int32MulWithOverflow on arm64 uses a cmp to set flags rather than > the multiply instruction itself, thus we can use a left shift when > the multiplication is by a power of two. > > This provides 0.15% for Speedometer2 on a Neoverse-N1 machine, > with React being improved by 0.45%. > > Change-Id: Ic8db42ecc7cb14cf1ac7bbbeab0e9d8359104351 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829472 > Commit-Queue: George Wort <george.wort@arm.com> > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82499} Change-Id: I896530a53fbdf6d397922124abddda4140144448 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854222Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: George Wort <george.wort@arm.com> Cr-Commit-Position: refs/heads/main@{#82696}
-
Leon Bettscheider authored
This CL adds a soft limit (via AllocationObserver) to run incremental marking for MinorMC. Once the soft limit is triggered, roots are marked. This a stepping stone for concurrent marking (YoungGenerationConcurrentMarkingVisitor, go/YGCMV) integration. Bug: v8:13012 Change-Id: I5bc9aeb80511159561845deb494023ade3fb7365 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3824339Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Leon Bettscheider <bettscheider@google.com> Cr-Commit-Position: refs/heads/main@{#82695}
-
Dominik Inführ authored
Objects in the from page could be promoted into the shared heap as well. While this shouldn't happen for references into evacuation candidates, I think it's easier to understand when there is a single conditional branch at the end. Bug: v8:13227, v8:11708 Change-Id: I999f10228ed5fdd70675a6d9c1e178eb152f39f0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854502Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/main@{#82694}
-
Michael Lippautz authored
This is a reland of commit 2115ba50. Adds flags to allow overriding marking support. This adds compatibility with EmbedderHeapTracer which allows for disabling incremental marking support with `--no-incremental-marking-wrappers`. The corresponding CppHeap flags are * `--cppheap-incremental-marking` * `--cppheap-concurrent-marking` This allows embedders that use types that do not support incremental and concurrent marking to switch from EmbedderHeapTracer to CppHeap. Bug: v8:13207 Change-Id: I43a47d7d035bff5d4b437c5bf01336a895b61217 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3851543Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#82693}
-
Derek Gonyeo authored
Replace `v8_unittests.cmx` with `v8_unittests.cml`, thus migrating the tests to CFv2. Bug: chromium:1256503 Change-Id: Iab90b406973585a513bce5a1a0884860c7c88e90 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3826050Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Wez <wez@chromium.org> Commit-Queue: Derek Gonyeo <dgonyeo@google.com> Cr-Commit-Position: refs/heads/main@{#82692}
-
Matthias Liedtke authored
The StructProxy::Create() used the static type information to inspect the value. However, for abstract references like anyref, dataref, ... this does not contain the required struct_index. To fix this the WasmTypeInfo stores the type_index for structs and arrays. Bug: v8:7748 Change-Id: I6e1af054711ada5e12c08949c125007e8185e486 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3850296 Commit-Queue: Matthias Liedtke <mliedtke@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> Cr-Commit-Position: refs/heads/main@{#82691}
-
Dominik Inführ authored
Concurrent markers could add work into the worklist before the CHECK. Bug: v8:12775, v8:13223 Change-Id: I8ac252b0fec8e5acbcfec56dad04830e596c709d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3854496 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#82690}
-
Leszek Swirski authored
This reverts commit 63b37c0e. Reason for revert: Seems to regress performance Original change's description: > Reland "[ptr-compr][x64][compiler] Support load map in compressed > form" > > This is a reland of commit 6ca3adb9 > > Fix build failed with V8_MAP_PACKING. > > Original change's description: > > [ptr-compr][x64][compiler] Support load map in compressed form > > > > ...to allow pointer decompression at use-site. > > > > Bug: v8:13056, v8:7703 > > Change-Id: If369286814c76340a945cc2a9fd863888a813080 > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3811737 > > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > > Commit-Queue: Hao A Xu <hao.a.xu@intel.com> > > Cr-Commit-Position: refs/heads/main@{#82242} > > Bug: v8:13056, v8:7703 > Change-Id: Ic753558058f70f6ee7850019aac9235b87d0e56a > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3815779 > Commit-Queue: Hao A Xu <hao.a.xu@intel.com> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82322} Bug: v8:13056, v8:7703 Change-Id: I8693af9189e214ec54a56149e0b29038e85838c2 Fixed: chromium:1352384 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3842931 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#82689}
-