- 29 Nov, 2021 14 commits
-
-
Samuel Groß authored
This low-level API implements virtual address space reservations on the supported platforms. An AddressSpaceReservation supports similar functionality as the global page management APIs in the OS class but operates inside a continuous region of previously reserved virtual address space. A reservation is backed by regular mmap mappings on Posix, by placeholder mappings on Windows, and by VMARs on Fuchsia. Bug: chromium:1218005 Change-Id: I99bc6bcbc26eb4aa3b54a31c671c9e06e92c471b 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/+/3270540 Commit-Queue: Samuel Groß <saelo@chromium.org> Reviewed-by: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78130}
-
Marja Hölttä authored
We had IsJSHandlerOnTop and IsExternalHandlerOnTop, which were almost opposites but not quite. We often did the same computation repeatedly for determining which kind of a handler is at the top (if any). This CL simplifies the logic, and only does the three-way logic once: either there's an external handler, a JS handler, or neither. It also removes dead code from Isolate::ReportPendingExceptions: we already do an early return if there's a JS handler on top, so we don't need to re-check. Bug: v8:12437 Change-Id: Ic15675bf2177772037d9fcec31c79019e4f0e02c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302802Reviewed-by: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#78129}
-
Dominik Inführ authored
This method should make it easier to debug memory leaks or out-of-memory errors with d8. Bug: v8:12198 Change-Id: I66d0ce8e04732badb7902453a1cd95ba9c29f3e4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303801Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/main@{#78128}
-
Benedikt Meurer authored
With https://crrev.com/c/3272577 we introduced a `StackFrame` cache for the inspector, which is keyed on the script ID, line and column number, so the syntactic properties of the function. However, the name that we report for functions is dynamic and can change (per closure) by explicitly reconfiguring the "name" property via ```js var f = function() { /* ... */ } Object.defineProperty(f, "name", {value: "super duper function"}); ``` for example, so we need to take that into account as well, and only use the cached `StackFrame` instance if the dynamic names still match up. Otherwise we just overwrite the cached entry with a new instance (the assumption here is that "name" isn't reconfigured often). Fixed: chromium:1274529 Bug: chromium:1268436 Change-Id: I519017c762aed5b4f93b9dc4553fa81d5979f1a1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3306376 Commit-Queue: Benedikt Meurer <bmeurer@chromium.org> Auto-Submit: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by: Simon Zünd <szuend@chromium.org> Cr-Commit-Position: refs/heads/main@{#78127}
-
QiuJi authored
{WasmInternalFunction::external} might return a function that is not a WasmExportedFunction, at which point the code in ProcessTypeFeedback fails. See crrev.com/c/3277878 for context. Bug: v8:12436 Change-Id: I447710cfa2dbdb64cba27922da85871d18bc79a3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303613Reviewed-by: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#78126}
-
Nikolaos Papaspyrou authored
Until now, histograms associated with isolate counters were created at the time of isolate initialization. This is too early because it happens before persistent memory is configured. Histograms created before persistent memory is set up are retrieved periodically but infrequently and are not "flushed" on process termination. As a result, a lot of samples from V8 are lost. This CL implements lazy creation of counter histograms, the first time that they are used. Bug: chromium:1270428 Change-Id: I8540b50b6c3dde1f477853a011b6c3f2c2c6ef9d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3284888 Commit-Queue: Nikolaos Papaspyrou <nikolaos@chromium.org> Reviewed-by: Omer Katz <omerkatz@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78125}
-
Michael Lippautz authored
This is a reland of 76f6c276 Original change's description: > cppgc: Fix data race ObjectSizeTrait > > Fix benign race in > https://clusterfuzz.com/testcase-detail/5203237072076800 > > Change-Id: I558b230e4905a48342d8e7cf70d39be5a1b7fdb8 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3306375 > Commit-Queue: Michael Lippautz <mlippautz@chromium.org> > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Auto-Submit: Michael Lippautz <mlippautz@chromium.org> > Reviewed-by: Omer Katz <omerkatz@chromium.org> > Cr-Commit-Position: refs/heads/main@{#78121} Change-Id: Ifa50f35591b2ae40f11a384f0fb2ff50115b2511 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3306379 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Auto-Submit: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78124}
-
Manos Koukoutos authored
To trigger speculative inlining, a function needs to run a few times on Liftoff-tier, and then tier-up to Turbofan. We make sure this happens by enabling --wasm-dynamic-tiering with a small budget, and running the critical functions until {%IsLiftoffFunction} is false. We also add an additional test. Bug: v8:12166 Change-Id: I6c5e0d3f1e3ba0df8510ae6850d8e9af1d01c179 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3306372Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/main@{#78123}
-
Maya Lekova authored
This reverts commit 76f6c276. Reason for revert: Mac64 ASAN is unhappy, please see https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Mac64%20ASAN%20-%20builder/194/overview Original change's description: > cppgc: Fix data race ObjectSizeTrait > > Fix benign race in > https://clusterfuzz.com/testcase-detail/5203237072076800 > > Change-Id: I558b230e4905a48342d8e7cf70d39be5a1b7fdb8 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3306375 > Commit-Queue: Michael Lippautz <mlippautz@chromium.org> > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Auto-Submit: Michael Lippautz <mlippautz@chromium.org> > Reviewed-by: Omer Katz <omerkatz@chromium.org> > Cr-Commit-Position: refs/heads/main@{#78121} Change-Id: I96c40a1e3421f59cf97efd4a844a041280989171 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3306377 Auto-Submit: Maya Lekova <mslekova@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Owners-Override: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/main@{#78122}
-
Michael Lippautz authored
Fix benign race in https://clusterfuzz.com/testcase-detail/5203237072076800 Change-Id: I558b230e4905a48342d8e7cf70d39be5a1b7fdb8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3306375 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Auto-Submit: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78121}
-
Ma Aiguo authored
Loong64 supports 4K-64K OS pages Fix loong64 unittests PlatformUsesGuardPages failure Change-Id: I1451685828ef1d857b7d2af3f1810286f84bdc50 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3299672Reviewed-by: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Cr-Commit-Position: refs/heads/main@{#78120}
-
Tobias Tebbi authored
Bug: chromium:1272026 Change-Id: Ic569c61b21d4c9c95d62e731a48076c871dd74ec Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303804Reviewed-by: Marja Hölttä <marja@chromium.org> Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/main@{#78119}
-
Kim-Anh Tran authored
Previously, we would encode 'other' as a reason for pausing when stepping too, however, it would not show as such in case it would overlap with another reason. This CL makes sure that we always report 'other' as a reason if we are stepping. Drive-by: only encode 'other' as a reason once Bug: chromium:1229541 Change-Id: Id73822dff68d1d54a2f1fafdf2a097e1377ece75 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3295346Reviewed-by: Jaroslav Sevcik <jarin@chromium.org> Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Kim-Anh Tran <kimanh@chromium.org> Cr-Commit-Position: refs/heads/main@{#78118}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/78cedba..2386a80 Rolling v8/third_party/zlib: https://chromium.googlesource.com/chromium/src/third_party/zlib/+log/f8964a5..2231b7c R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I3b3e7c679be02dd9165d2cd87986364547357dbb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3304144 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@{#78117}
-
- 27 Nov, 2021 2 commits
-
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/680d3fd..fe2e8aa Fix broken link to Setup/Teardown section (#1291) (Robert Adam) https://chromium.googlesource.com/external/github.com/google/benchmark/+/fe2e8aa R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: I0a70d832e9ac24565e3876fa80b9d60c5ff92150 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3304140 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@{#78116}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/26881c1..78cedba Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/e504863..665b74f Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/f51a154..c936d73 Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/3e0e32b..e2f3978 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/ace6575..11ea35f R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I9a4b3998bb75a848af479bedc5df36e3d8cd86f4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3304139 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@{#78115}
-
- 26 Nov, 2021 17 commits
-
-
Milad Fa authored
result needs to byte-swapped as it's currently in machine native order and Wasm requires LE ordering. Change-Id: I9c1d6752fbdaba18fb7a8d49a9848d04c1a21be8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303954Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Farazmand <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#78114}
-
Michael Lippautz authored
This allows the embedder to determine whether some function has been called from a destructor. See discussion in https://crrev.com/c/3302810 Bug: chromium:1273928 Change-Id: Icb5d98eff777574488a7d6de5e693c502c2fb53e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303793Reviewed-by: Omer Katz <omerkatz@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78113}
-
Milad Fa authored
Fixing a dcheck compilation error missed by https://crrev.com/c/3302850 Change-Id: I98c7394cbe64d99647656aebd175c8321f53c2de Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3300927Reviewed-by: Omer Katz <omerkatz@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#78112}
-
Michael Achenbach authored
This also fuzzes values of --budget-for-feedback-vector-allocation. Boundaries for the intervals are the default values in the code. No-Try: true Bug: v8:12434 Change-Id: I0a9d7421408a51c717c2edfe0e67c459f0a2834c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303792Reviewed-by: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Almothana Athamneh <almuthanna@chromium.org> Commit-Queue: Almothana Athamneh <almuthanna@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#78111}
-
Michael Lippautz authored
The CL converts uses of v8::internal::Worklist to heap::base::Worklist which does not require to know the number of tasks working with the work list upfront. heap::base::Worklist is the common implementation for V8's heap and cppgc and should be used/optimized going forward. Bug: v8:12426 Change-Id: If219edb8c0a8890c7bd0a131b8847b66256fdc21 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302799Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78110}
-
Michael Achenbach authored
Bug seems to not reproduce anymore as shown by: https://chromium-review.googlesource.com/c/v8/v8/+/3302803/1 No-Try: true Bug: v8:10243 Change-Id: Ic892b2c08d917f805ab6893e9dba86d112790abc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302803 Auto-Submit: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/main@{#78109}
-
Manos Koukoutos authored
See crrev.com/c/3277878 for context. We should only transform extenral to internal function references when passing a function value to a function-typed global or table. For their externref counterparts, we should preserve the reference unchanged. Bug: v8:11510, chromium:1273705 Change-Id: Ic1719c4d31e175f3a37ced6e4e4dfcd61a19ae57 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302790 Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#78108}
-
Michael Achenbach authored
This fuzzes more values of interrupt-budget on numfuzz. For now as a single instance. In a follow up we add it also to combined flags. No-Try: true Bug: v8:12434 Change-Id: I836c5e829ffeabfa4a4686d4d3d2fd43fce1ee88 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302797Reviewed-by: Almothana Athamneh <almuthanna@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#78107}
-
Liu Yu authored
Port commit 098f31f4 Bug: chromium:1052746 Change-Id: I4f9fd952c2ce8b51772eac89d4852d55363d1ed1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3292352 Auto-Submit: Liu yu <liuyu@loongson.cn> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/main@{#78106}
-
Takuto Ikuta authored
Bug: chromium:1208028 Change-Id: If24143b7d157561999e6003fe75fe9f9e386be85 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303612 Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#78105}
-
Igor Sheludko authored
... and thus avoid the need for special handling of objects located in external code space. This will also allow making HeapObject::IsBlah() checks faster when external code space is enabled. Bug: v8:11880 Change-Id: I12d07c05451ff198f0a6182d9b5849f76015e7fb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3300140Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#78104}
-
Nico Hartmann authored
This reverts commit 01a6b6d9. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Arm%20-%20debug/20669/overview Original change's description: > Reland "[wasm] Update spec tests" > > SIMD tests moved from the proposal directory to the main spec test > directory. Test expectations needed to be adjusted therefore. > > Change-Id: I02225da1f55ba173bbda22257edf45a3e987ba0d > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302795 > Commit-Queue: Andreas Haas <ahaas@chromium.org> > Reviewed-by: Maya Lekova <mslekova@chromium.org> > Cr-Commit-Position: refs/heads/main@{#78101} Change-Id: I7bdd6346e5ad593e7fe4289b5d6e15f9235ea26f No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302800 Auto-Submit: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Owners-Override: Nico Hartmann <nicohartmann@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#78103}
-
Michael Lippautz authored
Bug: chromium:1253650 Change-Id: I289dae157408ab68d76fb168aadf7924c3a65c09 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302798Reviewed-by: Anton Bikineev <bikineev@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78102}
-
Andreas Haas authored
SIMD tests moved from the proposal directory to the main spec test directory. Test expectations needed to be adjusted therefore. Change-Id: I02225da1f55ba173bbda22257edf45a3e987ba0d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302795 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/main@{#78101}
-
Nico Hartmann authored
This reverts commit d1538091. Reason for revert: https://ci.chromium.org/ui/p/chromium/builders/try/android-marshmallow-arm64-rel/1022301/overview Original change's description: > cppgc: Add temporary CHECK for diagnosing issue > > Bug: chromium:1253650 > Change-Id: I634501d5f092263ebd0f96826c79655c49ddce3b > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302792 > Reviewed-by: Anton Bikineev <bikineev@chromium.org> > Commit-Queue: Michael Lippautz <mlippautz@chromium.org> > Cr-Commit-Position: refs/heads/main@{#78096} Bug: chromium:1253650 Change-Id: I2e8ff73e8bac5379a86985f19558dbfa5caae94f No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302796 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Owners-Override: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/main@{#78100}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/d6ba952..680d3fd Add clang-tidy check (#1290) (dominc8) https://chromium.googlesource.com/external/github.com/google/benchmark/+/680d3fd R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: I1152e7dc79d41011a93a74e96711a62a9f6f2b78 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303154 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@{#78099}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/44b802e..26881c1 Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/038090f..f51a154 Rolling v8/third_party/aemu-linux-x64: R6v-hxhOQyZL9y_avAvE80v7II7utrI5Px4EUlMdtrAC..vRCm89BzABss-_H8vC-tLjcSf6uusZA9IBSSYtdw4_kC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/5d7ccbf..75423c3 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/afb82b2..2ffa1bd Rolling v8/tools/luci-go: git_revision:7972ced81286b7897c8712afe0f36eb9cf7bc098..git_revision:03f7c59feeac59c211e5b06fbbbc7405861b482e Rolling v8/tools/luci-go: git_revision:7972ced81286b7897c8712afe0f36eb9cf7bc098..git_revision:03f7c59feeac59c211e5b06fbbbc7405861b482e R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I16dd56334c813d03cdb2bf66f996e00966b47680 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3303151 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@{#78098}
-
- 25 Nov, 2021 7 commits
-
-
Ben Noordhuis authored
PTHREAD_STACK_MIN is an alias for __sysconf(__SC_THREAD_STACK_MIN_VALUE) in glibc 2.34. __sysconf() returns long, causing a -Werror,-Wsign-compare error build error. Change-Id: I15da8e7ee57a6979682ff7166990698965481586 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3301464 Commit-Queue: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78097}
-
Michael Lippautz authored
Bug: chromium:1253650 Change-Id: I634501d5f092263ebd0f96826c79655c49ddce3b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302792Reviewed-by: Anton Bikineev <bikineev@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78096}
-
Milad Fa authored
After https://crrev.com/c/3301477 compilation with gcc could fail with the following errors: ``` error: reference to 'MarkingType' is ambiguous error: reference to 'SweepingType' is ambiguous ``` Change-Id: Idb0f52853d377ec0c0f373f2e908e6c7fa0da914 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302850Reviewed-by: Omer Katz <omerkatz@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#78095}
-
Junliang Yan authored
Change-Id: Ie2815722ae1507cc5e2e7d510148063df3ebbf32 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302848Reviewed-by: Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/main@{#78094}
-
Dominik Inführ authored
ShouldVisit() uses obj.Size() to increment the live bytes counter after the object was successfully marked grey. However, this re-reads the length field which could have already been overwritten by a concurrent left-trimming operation on the main thread. Fix this by calculating the object size later with the length field we read before marking that object black. That value is guaranteed to be a SMI. Bug: chromium:1273352 Change-Id: I47e5a2df3eef61b4ef07af943f30123e5c2f7f9d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302793 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78093}
-
Omer Katz authored
This is a reland of 6747144c Original change's description: > cppgc: Parallel marking in atomic pause > > Bug: v8:12424 > Change-Id: I0633e1bd8c890c14ce2c5519253a5e6eb7592f04 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3295580 > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > Cr-Commit-Position: refs/heads/main@{#78081} Bug: v8:12424 Change-Id: I66a030b4e66647a76bbe3d114785d3052358b403 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3301477Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/main@{#78092}
-
Victor Gomes authored
The bytecode can be flushed and/or the function can be compiled and installed before the interruption to install the new batch of compiled baseline code. Bug: v8:12415, v8:12054 Change-Id: I092c8e0f31735119e6b2c6c46ce97ba8a4cdc91e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3302788Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/main@{#78091}
-