- 16 Mar, 2022 5 commits
-
-
Clemens Backes authored
This reverts commit f01a6098. Reason for revert: Consistent TSan failures: https://crbug.com/v8/12713 Original change's description: > [baseline] Enable concurrent sparkplug > > Bug: v8:12054 > Change-Id: I1b0ee1a9541a75412b882b259e421aa6e88317da > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3259658 > Auto-Submit: Victor Gomes <victorgomes@chromium.org> > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Commit-Queue: Toon Verwaest <verwaest@chromium.org> > Cr-Commit-Position: refs/heads/main@{#79485} Bug: v8:12054 Change-Id: Ide6bbac8c280a1373c5082bbe5356a205f3396c5 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3528496 Auto-Submit: Clemens Backes <clemensb@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#79492}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/18a1f19..3d95907 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/63a615b..b3fe2c1 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/3481645..b199f54 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/85f3c90..b60d34c R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I5d2149bbec079a8127089d33f38f64f952e9e9cd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3526718 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@{#79491}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/icu: https://chromium.googlesource.com/chromium/deps/icu/+log/1fa4e39..8a5b728 Add af and ur to common ICU (Trevor Perrier) https://chromium.googlesource.com/chromium/deps/icu/+/8a5b728 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,ftang@chromium.org Change-Id: I313ca3e905c578ecaa6a55916f9011fb67516e6c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3526720 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@{#79490}
-
Jakob Gruber authored
This CL implements most parts of the concurrent maglev pipeline. - MaglevConcurrentDispatcher: controls concurrent jobs. - MaglevCompilationInfo: holds job-global data, controls handle fiddling between the main isolate and local isolates, owns job-global state like the Zone. - MaglevCompilationUnit: same as before, holds per-unit data. Still missing: job finalization. Bug: v8:7700 Change-Id: I281178d945e79a0ba97fa2ac7023285d84a16641 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3516036Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#79489}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/9a71e5d..5704cd4 add note about wheels action to releasing docs (Dominic Hamon) https://chromium.googlesource.com/external/github.com/google/benchmark/+/5704cd4 Change artifact download name to dist to match upload name (#1371) (Nicholas Junge) https://chromium.googlesource.com/external/github.com/google/benchmark/+/3eab985 Remove conditional trigger from PyPI upload job (#1370) (Nicholas Junge) https://chromium.googlesource.com/external/github.com/google/benchmark/+/7d6f153 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: I85cd3a461f2a677973064ca51963e040d3cff8dd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3526719 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@{#79488}
-
- 15 Mar, 2022 16 commits
-
-
Samuel Groß authored
This is necessary due to the indirect call to memfd_create. The function is only used for testing though. Bug: v8:12682 Change-Id: Ifb9fd855213064165fa4c2125846ff592368b625 No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3525534Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Samuel Groß <saelo@chromium.org> Auto-Submit: Samuel Groß <saelo@chromium.org> Cr-Commit-Position: refs/heads/main@{#79487}
-
Samuel Groß authored
The use of memfd_create causes V8 to require glibc 2.27 which wasn't previously needed. This CL rewrites the affected code to check at runtime whether memfd_create is available and otherwise use mkstemp. Bug: v8:12682 Change-Id: I84dc3f5ab7504cec2b599bc92501ddecc2ae22cf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3516870Reviewed-by: Clemens Backes <clemensb@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Samuel Groß <saelo@chromium.org> Cr-Commit-Position: refs/heads/main@{#79486}
-
Victor Gomes authored
Bug: v8:12054 Change-Id: I1b0ee1a9541a75412b882b259e421aa6e88317da Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3259658 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/main@{#79485}
-
Milad Fa authored
Port 8a0d1b6f Original Commit Message: Modernise the RegList interface to be a proper class, rather than a typedef to an integer, and add proper methods onto it rather than ad-hoc bit manipulation. In particular, this makes RegList typesafe, adding a DoubleRegList for DoubleRegisters. The Arm64 CPURegList isn't updated to use (or extend) the new RegList interface, because of its weird type-erasing semantics (it can store Registers and VRegisters). Maybe in the future we'll want to get rid of CPURegList entirely and use RegList/DoubleRegList directly. R=leszeks@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I997156fe4f4f2ccc40b2631d5cb752efdc8a5ad2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3525084Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Farazmand <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#79484}
-
Al Muthanna Athamina authored
Bug: chromium:1306416 Change-Id: I103602ed1bea71d79a17a9a37c7eaf198575d371 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3521944Reviewed-by: Michael Achenbach <machenbach@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Almothana Athamneh <almuthanna@chromium.org> Cr-Commit-Position: refs/heads/main@{#79483}
-
Clemens Backes authored
It turned out that on arm and arm64 we over-estimated the code size of a Wasm module quite a bit. This CL adds some more output for the --trace-wasm-compilation-times flag, and adds a script to compute the factors we use for code size estimates from that output. I ran the script on a few benchmarks (an older Epic module, the current Photoshop module, and the benchmark from the linked bug), and adjusted the constants accordingly. Also, simplify the API of {ReservationSize} to only return a single number, and fail internally if we need to allocate more than the engine supports (which would only fail for artificially large modules). R=jkummerow@chromium.org Bug: chromium:1302310 Change-Id: I5b2c27ff3e360fb6738cf5dd697bcee09e106b6d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3522067Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#79482}
-
Maya Lekova authored
This CL adds a missing header that was implicitly included by "test/cctest/wasm/wasm-run-utils.h", which resulted in test failure with v8_enable_webassembly = true. Drive-by: Improve traceability of the fastcall cctests. Bug: chromium:1052746 Change-Id: I4cafbce2e390958befee896782ad13a9bdfc0f30 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3525113Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/main@{#79481}
-
Tamer Tas authored
R=machenbach@chromium.org,alexschulze@chromium.org,almuthanna@chromium.org,liviurau@chromium.org Bug: chromium:1298869 Change-Id: Ia08f5069bacf5134ba56265d64eff527d7dd96fb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3525134 Commit-Queue: Tamer Tas <tmrts@chromium.org> Auto-Submit: Tamer Tas <tmrts@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#79480}
-
Victor Gomes authored
Fix compilation issue in ia32. Change-Id: I7603a3bcebda0faf4e44d02e6c22601f7531a7aa Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3516746 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/main@{#79479}
-
Victor Gomes authored
Bug: v8:12709 Change-Id: Ia353ce364546fc34ca6b93a9873a20630a8085a5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3523045 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Manos Koukoutos <manoskouk@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/main@{#79478}
-
Tamer Tas authored
Format strings are the convention in Py3 R=machenbach@chromium.org,alexschulze@chromium.org Bug: chromium:1298869 Change-Id: Ia660e933f07d2a592c2edadaa99df68c47adb80d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3521789 Auto-Submit: Tamer Tas <tmrts@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#79477}
-
Michael Achenbach authored
No-Try: true Bug: chromium:1292013 Change-Id: Id9966157d28528b28e820d328b4941287a310209 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3521790Reviewed-by: Liviu Rau <liviurau@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#79476}
-
haoyuintel authored
This CL fixed cmpq in TestMoveSmi by using cmp_tagged and further optimizes Move(Register, Smi) for 31-bit negative Smi. To move a Smi of -1 to register, the disassembly before the commit is as: 48c7c0feffffff REX.W movq rax, 0xfffffffe The disassembly after the commit is as: b8feffffff movl rax, 0xfffffffe Bug: v8:12696 Change-Id: I6fafeec7959491ba8b084acf797c58910c2928fa Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3514654Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Haoyu Zhang <haoyu.zhang@intel.com> Cr-Commit-Position: refs/heads/main@{#79475}
-
Clemens Backes authored
This reverts commit 6e2c9bb2. Reason for revert: Fails layout test: external/wpt/wasm/serialization/arraybuffer/transfer.window.html Original change's description: > [serialize] copy bytes for non detachable array_buffer > in WriteJSArrayBuffer when array_buffer is not in > array_buffer_transfer_map_ > > According to https://html.spec.whatwg.org/multipage/structured-data.html#structuredserializeinternal > steps 13.3.2-4, should normally serialize array buffer which > is not detachable > > Bug: v8:12703 > Change-Id: I4554c5d07ae85e1a96a728ebba04c6a071575f6f > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3518910 > Reviewed-by: Marja Hölttä <marja@chromium.org> > Commit-Queue: Marja Hölttä <marja@chromium.org> > Cr-Commit-Position: refs/heads/main@{#79466} Bug: v8:12703 Change-Id: I548ab191ab7d0f8fa699958396a5e32e34d39568 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3524742 Auto-Submit: Clemens Backes <clemensb@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#79474}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/4f77cf9..9a71e5d Add BENCHMARK_STATIC_DEFINE to the Python bindings' `cc_binary` local defines. (#1369) (Nicholas Junge) https://chromium.googlesource.com/external/github.com/google/benchmark/+/9a71e5d Update LICENSE file to clearly state which file needs BSD 3 (#1366) (Vy Nguyen) https://chromium.googlesource.com/external/github.com/google/benchmark/+/4a1943d R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: Ib3d07292d03783c5c473df650011c4acb867449a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3524089 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@{#79473}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/5d2dc59..18a1f19 Rolling v8/buildtools/linux64: git_revision:f27bae882b2178ccc3c24f314c88db9a34118992..git_revision:bd99dbf98cbdefe18a4128189665c5761263bcfb Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/289d52c..a897d0f Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/b326cac..63a615b Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/c848a4e..3481645 Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/ae1b7ad..ae5e06d Rolling v8/third_party/zlib: https://chromium.googlesource.com/chromium/src/third_party/zlib/+log/85bdd64..b0676a1 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I7178731823e068efe7bd23b862d60b4f4f701bc6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3524088 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@{#79472}
-
- 14 Mar, 2022 16 commits
-
-
Milad Fa authored
Port: c69b0c80 Change-Id: Ie29079f976c7ab6fa14e5edc1da296a9e75b1bcf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3523822Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Farazmand <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#79471}
-
Nikolaos Papaspyrou authored
This CL moves the call to GCTracer::StopCycle for the full GC from Heap::CompleteSweeping full, which is called to force sweeping to finish, to GCTracer::NotifySweepingComplete, which is called as soon as sweeping finishes --- and symmetrically to a new method GCTracer::NotifyCppGCCompleted, which is called as soon as sweeping of the managed C++ heap finishes. In this way, a full GC cycle is reported as soon as sweeping is finished both for the V8 and the C++ managed heap. The changes introduced in this CL are essentially a partial revert of https://crrev.com/c/3456563, fixed in such a way that when the full GC cycle is reported, the current tracer event will be the correct one corresponding to that cycle. Bug: v8:12503 Bug: chromium:1154636 Change-Id: Icea07cf35a9565994e798b0500e9da72cd95f9ac Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3497318Reviewed-by: Omer Katz <omerkatz@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Nikolaos Papaspyrou <nikolaos@chromium.org> Cr-Commit-Position: refs/heads/main@{#79470}
-
Darius Mercadier authored
This reverts commit 3d5d99ff. Reason for revert: causes this crash: https://bugs.chromium.org/p/chromium/issues/detail?id=1303458 Original change's description: > [compiler] let InstructionSelector duplicate branch conditions > > Bug: v8:12484 > Change-Id: I44c2028efadbd70e7711f01d107995e0462f05d4 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3477094 > Reviewed-by: Tobias Tebbi <tebbi@chromium.org> > Commit-Queue: Darius Mercadier <dmercadier@chromium.org> > Cr-Commit-Position: refs/heads/main@{#79239} Bug: chromium:1303458, v8:12484 Change-Id: I129467bcb2507f2fba894f5dd58304eb139f739c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3522069Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Darius Mercadier <dmercadier@chromium.org> Cr-Commit-Position: refs/heads/main@{#79469}
-
jameslahm authored
strict parameters error before parse parentheses expression in ParsePrimaryExpression clear last next_arrow_function_info tracked strict parameters error, avoid throw syntax error when parse arrow function nested in a parentheses expression. Bug: v8:12688 Change-Id: Ib190ff5e04c9a83329c59421e9dd44f5a5907b07 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3516729Reviewed-by: Toon Verwaest <verwaest@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/main@{#79468}
-
Victor Gomes authored
Bug: v8:7700 Change-Id: Ib3b3f453e162e0913b8077ac8f8b9e2273deaed9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3521783 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#79467}
-
jameslahm authored
in WriteJSArrayBuffer when array_buffer is not in array_buffer_transfer_map_ According to https://html.spec.whatwg.org/multipage/structured-data.html#structuredserializeinternal steps 13.3.2-4, should normally serialize array buffer which is not detachable Bug: v8:12703 Change-Id: I4554c5d07ae85e1a96a728ebba04c6a071575f6f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3518910Reviewed-by: Marja Hölttä <marja@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#79466}
-
Milad Fa authored
Running `OS::AllocatePageSize()` on PPC simulator (which runs on x64 Linux) will make a `sysconf(_SC_PAGESIZE)` call which returns the native x64 Linux page size and not the PPC specific page size. This can cause a problem if used along side the current value of `kPageSizeBits` which is set to use PPC value even on Sim. As an example a newly added DHCECK under https://crrev.com/c/3497363 was failing on PPC Sim as a call to `AllocatableMemoryInDataPage` was being made which used kPageSizeBits on Sim. Meanwhile another function was using the `page_size_bits` value which gets set by _SC_PAGESIZE. The mismatch caused an incorrect final value being generated which failed the DCHECK. Change-Id: Iaf2d9cff16bbe5f6be5f3ec6a13b99be94776f48 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3519758Reviewed-by: Junliang Yan <junyan@redhat.com> Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Milad Farazmand <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#79465}
-
Marja Hölttä authored
The fuzzers were passing the flag --es-staging which doesn't exist. This CL updates them to pass the flag --harmony which does exist. Change-Id: I02c83026e5b9bdf49e51e700f16702bf56cd49e8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3522064Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#79464}
-
Toon Verwaest authored
If a value is in multiple registers, pick one of them to be freed. Bug: v8:7700 Change-Id: I1886b977187b4d8e939ff106edde4ccf716661d7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3522063 Auto-Submit: Toon Verwaest <verwaest@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#79463}
-
Victor Gomes authored
The CL implements binary operations bytecodes as generic nodes that call the correspondent runtime builtin. Bug: v8:7700 Change-Id: I82c5e20e4103d4ef367184af1242bae7f7f93fe0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3509392Reviewed-by: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/main@{#79462}
-
jameslahm authored
- inline Array.prototype.indexOf in js-call-reducer - inline Array.prototype.includes in js-call-reducer Bug: v8:12390 Change-Id: Idb5669da3019f0f56af0084fccd1d616d4c5098e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3473994Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Marja Hölttä <marja@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#79461}
-
Leszek Swirski authored
Modernise the RegList interface to be a proper class, rather than a typedef to an integer, and add proper methods onto it rather than ad-hoc bit manipulation. In particular, this makes RegList typesafe, adding a DoubleRegList for DoubleRegisters. The Arm64 CPURegList isn't updated to use (or extend) the new RegList interface, because of its weird type-erasing semantics (it can store Registers and VRegisters). Maybe in the future we'll want to get rid of CPURegList entirely and use RegList/DoubleRegList directly. Change-Id: I3cb2a4d386cb92a4dcd2edbdd3fba9ef71f354d6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3516747 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#79460}
-
Samuel Groß authored
Instead of implementing ExternalObjects as plain JSObjects with a single EmbedderDataSlot pointing to a Foreign containing the actual raw pointer, this CL now creates a new JSExternalObject type that directly contains the external pointer. As a side-effect of this refactoring, nullptr values are now no longer valid for ExternalObjects. Change-Id: Ic8ff334681c966e823ca70f34dd1efaaa21a0789 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/+/3513234Reviewed-by: Igor Sheludko <ishell@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Samuel Groß <saelo@chromium.org> Cr-Commit-Position: refs/heads/main@{#79459}
-
Marja Hölttä authored
Bug: v8:11525 Change-Id: I582f17d6ecfa47394bc29692f788c5119aac5ce9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3516745Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#79458}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/43a4f0e..5d2dc59 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/271796d..b326cac R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I1c8b08d850885bbadbd471a770211f1e98f8ba21 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3517971 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@{#79457}
-
JianxiaoLuIntel authored
This CL fixes a deopt loop that might happen in case of mixing Number with BigInt in bitwise binary operations. Bug: v8:12693 Change-Id: Ib6a08d0c74a954ade3719bd6bd49ca2988d88e69 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3505542Reviewed-by: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Jianxiao Lu <jianxiao.lu@intel.com> Cr-Commit-Position: refs/heads/main@{#79456}
-
- 13 Mar, 2022 2 commits
-
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/705202d..4f77cf9 Fix float comparaison and add float comparison warning (#1368) (Bensuperpc) https://chromium.googlesource.com/external/github.com/google/benchmark/+/4f77cf9 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: Ibe4d4953920ea1415fadaca1a636750f9590c881 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3520366 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@{#79455}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/6bce1d6..43a4f0e R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I03b8c3002da073209e092ba1204646ad7f856aeb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3520362 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@{#79454}
-
- 12 Mar, 2022 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/797674d..6bce1d6 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I9f375f81935d563070e636468cf483b9fff66f46 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3517461 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@{#79453}
-