- 20 Nov, 2021 3 commits
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/994f319..3335020 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/81212d2..a63bbc1 Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/7f068e5..038090f Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/e6e1eb6..8b575c9 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/1c7dec3..b6c1ed4 Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/1a3e2a2..3e0e32b Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/8caa4c5..b0d8d05 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I693d6a3bdac5fc1265cfa24eca70dd13dac04fa6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3294008 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#78006}
-
Frank Tang authored
Remove two flags for feature shipped in m95 --harmony_intl_displaynames_v2 --harmony_intl_more_timezone Bug: v8:11637, v8:11661 Change-Id: I9432865722664f64c64b5bf7fde5e05e6b324b20 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3292518Reviewed-by: Shu-yu Guo <syg@chromium.org> Commit-Queue: Frank Tang <ftang@chromium.org> Cr-Commit-Position: refs/heads/main@{#78005}
-
Ng Zhi An authored
Bug: v8:12284 Change-Id: Iaa2b98029b46e5d22fac19bf2db5abd56f82670b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3272641Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/main@{#78004}
-
- 19 Nov, 2021 14 commits
-
-
Ng Zhi An authored
This is a partial revert of https://crrev.com/c/2033171 to bring back WasmModuleObject::Compile, that compiles Wasm bytes into a Wasm module. Also adding a simple test to make sure the compile works. There are some users, e.g. Cloudflare, Halide, see bug for more details. Bug: v8:10461 Change-Id: I199abea108311025decad793d2232a08bba05d38 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3292088 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Reviewed-by: Adam Klein <adamk@chromium.org> Cr-Commit-Position: refs/heads/main@{#78003}
-
Milad Fa authored
PPC and S390 implement vsel slightly differently which is also reflected on the simulator. Change-Id: Iec5e49bfae09d8bf15149e5ab149b82971271b24 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3293409Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#78002}
-
Junliang Yan authored
Change-Id: I318a97a67242527c3a0f7a1dcd74b5503e5dcc55 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289836Reviewed-by: Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/main@{#78001}
-
Ng Zhi An authored
Bug: v8:12244,v8:12245 Change-Id: Ie2b1801d7535e142774a1d912b992a203b1b8ecc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3284005 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#78000}
-
Igor Sheludko authored
... in order to ease issues debugging. Bug: chromium:1241665 Change-Id: I3b51d3eaaa9c03cfadbdadc4e91dee35617f4b33 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3293090 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/main@{#77999}
-
Leszek Swirski authored
`foo ## __LINE__` just emits foo__LINE__ because of how preprocessor expansion works. The typical solution for this is to use a CONCAT macro, but we in fact already have a helper for what this is trying to solve, UNIQUE_IDENTIFIER, so just use that instead. Change-Id: Icea3f01db458c5d557e0affd3b004f4478c6c315 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3293084 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/main@{#77998}
-
Samuel Groß authored
Previously, an allocation from a separate thread could grab the just-released region and make it accessible before the regions permissions are changed to kNoAccess at the end of ReleasePages. Bug: v8:12414 Change-Id: I98c8f8e3df76d4a44c357ddab107cfeff20049b8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3293083Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Samuel Groß <saelo@chromium.org> Cr-Commit-Position: refs/heads/main@{#77997}
-
Dominik Inführ authored
When starting incremental marking from an allocation observer, the DCHECK for AllowGarbageCollection::IsAllowed() fails. We need to explicitly allow this here in order to use SafepointScope. Bug: chromium:1271659, v8:11708, v8:12413 Change-Id: I8607a82faed50f8dfffce1e495fb5e715a47bc1b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3293086 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Auto-Submit: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#77996}
-
Ray Wang authored
Bug: v8:12389 Change-Id: I53c728ab0c8ba38c7dd96c7e1089f771ba44b9f0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289227Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#77995}
-
Victor Gomes authored
Bug: v8:12054 Change-Id: I756c0a0d05860e2de748842157f4fa54726e94da Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289177 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#77994}
-
Victor Gomes authored
Bug: v8:12054 Change-Id: Ic121ef55ae7f8578aca3fc8303976f98959d2c8b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289175 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#77993}
-
Patrick Thier authored
std: :iterator is deprecated C++17. Use base::iterator instead. Change-Id: I682dc3b1547120e067a64eea4f71ffe4ee55ad16 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289168 Commit-Queue: Maya Lekova <mslekova@chromium.org> Auto-Submit: Patrick Thier <pthier@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/main@{#77992}
-
Ng Zhi An authored
This reverts commit 2f17c069. Reason for revert: related to --concurrent_sparkplug and not to this Original change's description: > Revert "heap: Fix compaction aborting all pages" > > This reverts commit 18236dcb. > > Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20no-concurrent-marking/6576/overview > > Original change's description: > > heap: Fix compaction aborting all pages > > > > In such a case, we must post-process all aborted pages. The early > > bailout assumed that not pages are aborted before trying compaction. > > > > Bug: chromium:1271229 > > Change-Id: I4c5998b9d1c7974f7d1d25a6729f42d5690a5759 > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289171 > > Reviewed-by: Hannes Payer <hpayer@chromium.org> > > Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> > > Commit-Queue: Michael Lippautz <mlippautz@chromium.org> > > Cr-Commit-Position: refs/heads/main@{#77987} > > Bug: chromium:1271229 > Change-Id: Ia4b8687a0f3742bf0c894989f68c249b6c87c446 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291900 > Auto-Submit: Zhi An Ng <zhin@chromium.org> > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> > Owners-Override: Deepti Gandluri <gdeepti@chromium.org> > Cr-Commit-Position: refs/heads/main@{#77988} Bug: chromium:1271229 Change-Id: Iba76a1ab11f1a52b845305f0aee4710af1406a91 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3292087 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/main@{#77991}
-
Milad Fa authored
Change-Id: I2abc916894b84aad5794e4a69979ca8683644806 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289933 Commit-Queue: Milad Fa <mfarazma@redhat.com> Reviewed-by: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/main@{#77990}
-
- 18 Nov, 2021 23 commits
-
-
Deepti Gandluri authored
This reverts commit 19bbd0f7. Reason for revert: Fails on future bot - https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20no-concurrent-marking/6576/overview Original change's description: > [baseline] Enable Concurrent Sparkplug on future > > Bug: v8:12054 > Change-Id: I7bbd22067a9e98038a08ce5cb4b0ef494aadd651 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289252 > Commit-Queue: Victor Gomes <victorgomes@chromium.org> > Commit-Queue: Milad Fa <mfarazma@redhat.com> > Auto-Submit: Victor Gomes <victorgomes@chromium.org> > Reviewed-by: Milad Fa <mfarazma@redhat.com> > Cr-Commit-Position: refs/heads/main@{#77986} Bug: v8:12054 Change-Id: I97a511b00a79a606ac9abb6d15b707c75706deee No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3292095 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/main@{#77989}
-
Zhi An Ng authored
This reverts commit 18236dcb. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20no-concurrent-marking/6576/overview Original change's description: > heap: Fix compaction aborting all pages > > In such a case, we must post-process all aborted pages. The early > bailout assumed that not pages are aborted before trying compaction. > > Bug: chromium:1271229 > Change-Id: I4c5998b9d1c7974f7d1d25a6729f42d5690a5759 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289171 > Reviewed-by: Hannes Payer <hpayer@chromium.org> > Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> > Commit-Queue: Michael Lippautz <mlippautz@chromium.org> > Cr-Commit-Position: refs/heads/main@{#77987} Bug: chromium:1271229 Change-Id: Ia4b8687a0f3742bf0c894989f68c249b6c87c446 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291900 Auto-Submit: Zhi An Ng <zhin@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Owners-Override: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/main@{#77988}
-
Michael Lippautz authored
In such a case, we must post-process all aborted pages. The early bailout assumed that not pages are aborted before trying compaction. Bug: chromium:1271229 Change-Id: I4c5998b9d1c7974f7d1d25a6729f42d5690a5759 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289171Reviewed-by: Hannes Payer <hpayer@chromium.org> Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#77987}
-
Victor Gomes authored
Bug: v8:12054 Change-Id: I7bbd22067a9e98038a08ce5cb4b0ef494aadd651 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289252 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#77986}
-
Clemens Backes authored
The entries contain more than just deoptimization info. Thus rename them to "entries" and use "EntryBuilder" instead of "DeoptimizationInfo". Drive-by: Remove the redundant {emitted_} field, just check if {offset_} was initialized. R=jkummerow@chromium.org Bug: v8:12401 Change-Id: Ifedc265dc27dd6817b731d9e24a1d8654edc99de Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291310 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#77985}
-
Junliang Yan authored
Change-Id: Ibd3ba4cd1edc174fdfaa826b8a6f7731f4627365 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289932Reviewed-by: Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/main@{#77984}
-
Clemens Backes authored
Verify certain properties when emitting safepoint tables: - entries are ordered by PC, - trampoline PCs are also in order, - all trampoline PCs come after all regular PCs, and - an entry with a deopt index also has a trampoline PC set. Drive-by: Slightly simplify logic in {UpdateDeoptimizationInfo}. Drive-by 2: Inline IsIdenticalExceptForPc into the only caller, and further simplify the logic there. R=jkummerow@chromium.org Bug: v8:12401 Change-Id: I8ea41b1628dceb914191f23165c8dccb79f9204c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289162 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#77983}
-
Michael Lippautz authored
We should not assume that the embedder can run with concurrent marking even though we just finalize the GC at this point. Bug: chromium:1271371 Change-Id: I3d3a45e4283dcae9c6fdd1a467158254914868c3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289169Reviewed-by: Omer Katz <omerkatz@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#77982}
-
Michael Lippautz authored
Change-Id: I4886923bd3b2c29f5f317fbfab078af4d5dd8510 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289156 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/main@{#77981}
-
Victor Gomes authored
Bug: v8:12054 Change-Id: I492ceb377eb0e38f249f76e9f2a5cb554781652c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291321 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#77980}
-
Clemens Backes authored
This removes the redundant {Safepoint::kNoDeoptimizationIndex} definition, and uses {SafepointEntry::kNoDeoptIndex} consistently. R=jkummerow@chromium.org Bug: v8:12401 Change-Id: I4e12862d96a2b97245f8dde62243013425e53ff1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289159Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#77979}
-
Milad Fa authored
Change-Id: Ie61638fbc61b5a84dc9ba396e4df7193a2ebd6a7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291547Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#77978}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/220a7fc..994f319 Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/b138e6c..075dd7e Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/707d75f..81212d2 Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/51ffc5e..7f068e5 Rolling v8/third_party/android_sdk/public: AuYa11pULKT8AI14_owabJrkZoRGuovL-nvwmiONlYEC..Ez2NWws2SJYCF6qw2O-mSCqK6424l3ZdSTpppLyVR_cC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/1a5ffbc..e6e1eb6 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/2efe3d7..1c7dec3 Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/1b18723..1a3e2a2 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/2c3af75..8caa4c5 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I08fb9b9716466ecd84e01873e2d3d9019dd28056 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3290473 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#77977}
-
Dominik Inführ authored
A SafepointScope might need to block for a shared GC initiated from another client isolate. This means that anytime we create a SafepointScope a shared GC may run. This CL adds a DCHECK to ensure AllowGarbageCollected::IsAllowed() holds for each SafepointScope. So far this DCHECK was only run in the less likely event that a SafepointScope actually runs a shared GC. Which is technically good enough but it is easy to miss use cases of SafepointScope where this does not hold. Bug: v8:11708, v8:12377 Change-Id: I30cc33c05ebe4835430e1d699a86079810523858 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289625Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/main@{#77976}
-
Clemens Backes authored
Many callers already pass an int, so there is no point in storing as an uint32_t internally. The style guide also recommends int as the general data type for integers, even if it's statically known that only positive values will occur. R=jkummerow@chromium.org Bug: v8:12401 Change-Id: I6067139f514895f925d1c536112b4cb5c2c24a36 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289157 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#77975}
-
Seth Brenith authored
In order to determine which module contains V8, v8windbg has been looking for the known symbol v8::Script::Run. However, that symbol might not be found if the function was inlined. To fix, we should instead choose a symbol that is declared with V8_NOINLINE. Change-Id: Ib73dfb27f35ab81ab69f92ffab944f6f1096b895 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3290107Reviewed-by: Maya Lekova <mslekova@chromium.org> Commit-Queue: Seth Brenith <seth.brenith@microsoft.com> Cr-Commit-Position: refs/heads/main@{#77974}
-
Jakob Gruber authored
Refactor s.t. we now compact the DependentCode weak fixed array both when adding new entries and when marked for deopts. Bug: v8:12397 Change-Id: I842f3369644e88cc0b98a1f1371259c920cc8bbf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291320 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Auto-Submit: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#77973}
-
Victor Gomes authored
The cpu profiler tests require a precise trace and become flaky when running with concurrent sparkplug. Bug: v8:12054 Change-Id: I62a20d15530c610bc1091371781c28687c718720 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289158 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#77972}
-
Manos Koukoutos authored
Using a map to store node states in Int64Lowering has proven slow. Therefore, we change the data structure to a vector indexed by node ids. Bug: v8:12166, chromium:1271179 Change-Id: I13b78091fe1a6a13c26afd706d3839b0c04390f4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291308Reviewed-by: Andreas Haas <ahaas@chromium.org> Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/main@{#77971}
-
Kim-Anh Tran authored
This CL explicitly encodes the 'other' reason for breaking for: * regular breakpoints * triggered pause events. The reason for explicitly encoding the reason is that we may otherwise not know why we pause when we handle it. This knowledge is needed in order to fully support instrumentation breakpoints, e.g. if we do not know that we paused on a triggered pause, and this happens to overlap with an instrumentation, we would previously only report 'instrumentation' as a reason which would be wrong. Bug: chromium:1229541 Change-Id: I93c08f965a491f6d34f280157b182a78d5b3cf07 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289638Reviewed-by: Jaroslav Sevcik <jarin@chromium.org> Commit-Queue: Kim-Anh Tran <kimanh@chromium.org> Cr-Commit-Position: refs/heads/main@{#77970}
-
Jakob Gruber authored
Prior to this CL we regularly generated high counts of code dependencies, and installation was not the most efficient. This CL 1) implements early dependency deduplication and 2) simplifies the way dependencies are persisted on the heap through DependentCode. Re 1): we dedupe twice, once based on the CompilationDependency contents, and again once we know the final target object. Re 2): Instead of a linked list of weak fixed arrays per dependency group, store deps in a flat array together with a bitset of their dependency groups. See also: https://docs.google.com/document/d/1B34S1s3Iv6hbquZ93RugD0b-ZKfHEptJ8Fk_YyOvjDk/edit Bug: v8:12195,v8:12397 Change-Id: I9ab47f6d87b10558194b5de30a36b1122f7e362a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3283074Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#77969}
-
Victor Gomes authored
When compiling code off-thread in the large code space, we need to register the chunk in the isolate using AddCodeMemoryChunk. In the main thread, this is done when AllocateRaw calls NotifyOldGenerationExpansion. Bug: chromium:1269315, v8:12054 Change-Id: I46ea5c9cdcd063708292dd49aac6d39091e11ba4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289151 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/main@{#77968}
-
Patrick Thier authored
Raise an exception if CallFunction is invoked with a class constructor. The check was initially removed in [1] but there are cases where we could end in CallFunction with class constructors from optimized code. [1] https://crrev.com/c/3186434 Bug: chromium:1271384 Change-Id: I0d700c4b1d117334c1c4c14719e24cd1f2c5e3a2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3291313 Commit-Queue: Patrick Thier <pthier@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#77967}
-