- 05 Apr, 2022 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/d162691..1356876 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/e9c9bdf..1850510 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/a9d86a4..c4e6210 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/a9c548e..c4b1b77 Rolling v8/third_party/zlib: https://chromium.googlesource.com/chromium/src/third_party/zlib/+log/d1aa7af..3b1df9d Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/c8e9f23..fbe0742 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: Ia2bed4e4a7ea13cd708471339269c7e4262abb9b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3571201 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@{#79768}
-
- 04 Apr, 2022 39 commits
-
-
Junliang Yan authored
Change-Id: Iadf5d379e105232c0992e3588ecd9a1c5f3500d2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3570422Reviewed-by:
Milad Farazmand <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/main@{#79767}
-
Anton Bikineev authored
The CL makes sure to extract and copy Oilpan young GC metrics to v8::metrics::GarbageCollectionYoungCycle. In addition, it makes sure that metrics are not reported twice by bailing out from GCTracer::NotifyCppGCCompleted() for young GC cycles (the metrics are reported later in Heap::CollectGarbage() by calling GCTracer::StopCycle()). Bug: chromium:1029379 Change-Id: I07bf51e85a76a7cdbeeb8d87c9072edf2634158b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3545168Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Anton Bikineev <bikineev@chromium.org> Cr-Commit-Position: refs/heads/main@{#79766}
-
Junliang Yan authored
Change-Id: I0b69c8042f1b88efc14d2923c595b14f45b9557e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568471Reviewed-by:
-
Junliang Yan authored
Change-Id: I698fbf3d49ed7e88a6004ead38427e4ca3a02663 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3569681Reviewed-by:
-
Bryant Chandler authored
cr_fuchsia_package is deprecated in favor of using the Fuchsia SDK provided rules directly. Bug: chromium:1092804 Change-Id: I86a59b6a717cb0aa8c3473e8410bdd98f0ffd042 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3537883Reviewed-by:
Victor Gomes <victorgomes@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Commit-Queue: Bryant Chandler <bryantchandler@chromium.org> Cr-Commit-Position: refs/heads/main@{#79763}
-
Jakob Gruber authored
.. to simplify logic within compiler.cc. GetOrCompileOptimized now only returns Code object if the requested optimized Code object is available. This change also required updating CompileLazy to install the appropriate Code object before potentially calling CompileOptimized_* runtime functions in order to satisfy the is_compiled precondition. Bug: v8:12161 Change-Id: I991dbcc0ba8f3d635aa1e1f06e4cffd89e08a47b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3562978Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Jakob Linke <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#79762}
-
Jakob Gruber authored
If we've already cached OSR'd code for the current function but with a different osr offset, fall back to synchronous compilation. This avoids degenerate cases where we repeatedly spawn OSR jobs but then fail to install them. Drive-by: More consistent --trace-osr output. Drive-by: Rename kCompileForOnStackReplacement to kCompileOptimizeOSR for name consistency. Drive-by: Add JSFunction::DebugNameCStr() for more convenient PrintF's. Bug: v8:12161 Change-Id: I2b4a65bc9e082d85d7048a3e92ef86b07d396687 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3560431Reviewed-by:
-
Bryant Chandler authored
cr_fuchsia_package is deprecated in favor of using the Fuchsia SDK provided rules directly. This CL adds a cmx file specifically for v8_unittests. CMX files define fuchsia components, see https://chromium-review.googlesource.com/c/chromium/src/+/3529652 for more info. Bug: chromium:1092804 Change-Id: Ibf1d866ec6b94a0e1a7a7c7c443a6ee80e3b1042 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3537885Reviewed-by:
-
Jakob Gruber authored
Tweak a few names, remove a few GetIsolate calls, other minor usability refactors. It may be worth taking a closer look at the impl in the future, currently the design choices don't seem ideal (see the added TODO on top of the class). The reland is unchanged from the original CL. Bug: v8:12161 Change-Id: I9971f7f2fb08b7a1ec2d57b2a0e4accdc11191ca Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568444Reviewed-by:
-
Clemens Backes authored
If the immediate is a 32-bit value, we can just write the lower half of the target register, the upper half will automatically be zero-extended. R=tebbi@chromium.org Bug: v8:10005 Change-Id: Ib3c54c9f6ac2434c7345c507529298233d6b7d6f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3563565Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#79758}
-
Leszek Swirski authored
Port the eager deopt handling in the use marker and register allocator to do the same thing with lazy deopts. This requires moving the lazy deopt info to be a pseudo-input before the node, same as eager deopt info, so that the regalloc can read it without needing the Node's opcode. For now, this means that a node cannot both eager- and lazy-deopt; if we need this in the future we can rethink it. Bug: v8:7700 Change-Id: I96292af9c483f285b1e45bfb374c8dc600fa6347 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568452Reviewed-by:
-
Joyee Cheung authored
Since the code is generated unconditionally, using a DCHECK to check that shared RO heap is enabled breaks builds with v8_enable_shared_ro_heap set to false, this patch turns that into a CSA_DCHECK so it only crashes when V8 actually attempts to store into a shared struct while the RO heap isn't shared at run time. Refs: https://github.com/nodejs/node/pull/42115 Bug: v8:12547 Change-Id: I30d9a02b98a0b647097125c0a9d141e40d6348cc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3561598Reviewed-by:
Shu-yu Guo <syg@chromium.org> Reviewed-by:
-
Dominik Inführ authored
Fail immediately when page allocation fails during deserialization. We would crash immediately in the GC following the allocation failure but with a less descriptive error message. Bug: v8:12514 Change-Id: I688d9bac5978ca7af3b24830999c992e1df32dce Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568458Reviewed-by:
-
Camillo Bruni authored
- Add references from CodeLogEntry to DeoptLogEntry - Add simple basic blocks in the disassembly code view Bug: v8:10644 Change-Id: I15f3b56751d515b902185b08f9454be3951ffa48 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3540142Reviewed-by:
Patrick Thier <pthier@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#79754}
-
Junliang Yan authored
Change-Id: Id3c0c09ca0c0e3b95ec51a8563d092c8410d3bc5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568925Reviewed-by:
-
Jakob Gruber authored
- Unhandlify OSROptimizedCodeCache::GetOptimizedCode. - Unstatic-fy FeedbackVector::SetOptimizedCode. - Remove frame-walking logic during the OSR tierup decision. The reland is unchanged from the original CL. Bug: v8:12161 Change-Id: Ibf03a9dd9a6fcd38c0664e5d5014a26d0240e035 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568463Reviewed-by:
-
Michael Achenbach authored
No-Try: true Bug: chromium:1245634 Change-Id: I1fca09f9e4917e9bb43ccc86a13220d1c13a3434 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3569024 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Liviu Rau <liviurau@chromium.org> Commit-Queue: Liviu Rau <liviurau@chromium.org> Cr-Commit-Position: refs/heads/main@{#79751}
-
Camillo Bruni authored
- Process the minidump in a separate function to avoid keeping references to the mmapped file during disposal - Clear all MinidumpReader variables before disposing the mmapped file Change-Id: I0ce468597329d6f7d703a08309e4be378d9c27cf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568469Reviewed-by:
-
Michael Achenbach authored
Bug: chromium:1292013 Change-Id: I5d73c8a500d66143e569d4605133948a71c82c8b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568468 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Benedikt Meurer authored
This changes the logic for generating method names in `error.stack` to prepend an inferred type name only when the function name is a valid ECMAScript identifiers and does not equal the inferred type name, to (1) give developers more control over the exact name shown in `error.stack`, as well as (2) avoid confusion in the presence of renaming of local variables. Previously we'd leave the function name as-is if it was prefixed by the inferred type name, but that condition is unnecessarily strict, and led to a bunch of inconsistencies around special names like `<instance_member_initializer>` where this dynamic approached often prefixed it with the correct type name, but also sometimes got it wrong and prepended `Object.`, which is very unfortunate and misleading. Specifically for these special names, we'll add logic later in the parser to infer a useful (complete) name. The design doc (https://bit.ly/devtools-method-names-in-stack-traces) contains more background and examples of why we do this change. Doc: https://bit.ly/devtools-method-names-in-stack-traces Fixed: chromium:1294619 Bug: chromium:1283435 Change-Id: Ib8b528ba25255dcd07e9d11044c562c11d699bcb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3565724Reviewed-by:Yang Guo <yangguo@chromium.org> Commit-Queue: Benedikt Meurer <bmeurer@chromium.org> Cr-Commit-Position: refs/heads/main@{#79748}
-
Milad Fa authored
Port 5a7ed57b R=ishell@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: Ie5abd86473a44fd38dc380836b08a644f1b21ab1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3569221Reviewed-by:
Igor Sheludko <ishell@chromium.org> Commit-Queue: Milad Farazmand <mfarazma@redhat.com> Reviewed-by:
Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/main@{#79747}
-
Jakob Gruber authored
This is a reland of commit 3ce690ee Changed for the reland: - Remove the currently-unused BytecodeArray member to avoid MSAN failures. - s/return/continue/ in optimizing-compile-dispatcher. Original change's description: > [osr] Basic support for concurrent OSR > > This CL adds basic support behind --concurrent-osr, > disabled by default. > > When enabled: > 1) the first OSR request starts a concurrent OSR compile job. > 2) on completion, the code object is inserted into the OSR cache. > 3) the next OSR request picks up the cached code (assuming the request > came from the same JumpLoop bytecode). > > We add a new osr optimization marker on the feedback vector to > track whether an OSR compile is currently in progress. > > One fundamental issue remains: step 3) above is not guaranteed to > hit the same JumpLoop, and a mismatch means the OSR'd code cannot > be installed. This will be addressed in a followup by targeting > specific bytecode offsets for the install request. > > This change is based on fanchen.kong@intel.com's earlier > change crrev.com/c/3369361, thank you! > > Bug: v8:12161 > Change-Id: Ib162906dd4b6ba056f62870aea2990f1369df235 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3548820 > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Commit-Queue: Jakob Linke <jgruber@chromium.org> > Cr-Commit-Position: refs/heads/main@{#79685} Bug: v8:12161 Change-Id: I48b100e5980c909ec5e79d190aaea730c83e9386 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3565720Reviewed-by:
-
Milad Fa authored
Port 1ff685d8 Original Commit Message: .. since they are the same as eager deopts (% an unused counter). R=jgruber@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I5575f2b14393e0b1a653b8d6be00c9dab338c160 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568924Reviewed-by:
Jakob Linke <jgruber@chromium.org> Commit-Queue: Milad Farazmand <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#79745}
-
Michael Achenbach authored
Bug: chromium:1292016 Change-Id: I9404ca1c38c6231cada6c5d9af5e5859e4c0e261 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568467 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Alexander Schulze authored
Bug: v8:12581 Change-Id: I4d98e48801ffcfbe507c61ba296da67359e3f5cb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568464 Auto-Submit: Alexander Schulze <alexschulze@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
jameslahm authored
- Add ConvertUtf8StringToUtf16 to convert utf8 encoded file path to utf16 encoded path on windows. - Add unicode filename support in OS::FOpen, OS::MemoryMappedFile::open and OS::MemoryMappedFile::create on windows. Bug: v8:12541 Change-Id: I65396c3211355e41e8952bc0587ff01bbb720f9e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3538284Reviewed-by:
Clemens Backes <clemensb@chromium.org> Reviewed-by:
-
Junliang Yan authored
Change-Id: I8605938213f0fe2b322e188e95aeb3dc22692923 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568862Reviewed-by:
-
Michael Achenbach authored
No-Try: true Bug: chromium:1288926 Change-Id: Ie28021a34649aeaa68c34252ffe181163a7df2ae Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568466 Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Igor Sheludko authored
Bug: v8:11880 Change-Id: Id3975d0c10ac5ece5c55d9db5ae7c6786fde2dfe Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3564566Reviewed-by:
-
Alexander Schulze authored
Bug: v8:12581 Change-Id: I2e869a81f6a8c785954a73cf2e97dae2a9ab3ade Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568450 Auto-Submit: Alexander Schulze <alexschulze@chromium.org> Reviewed-by:
-
Leszek Swirski authored
It doesn't take into account stack-slot liveness at time of spill, so it can cause false sharing. Bug: v8:7700 Change-Id: Ib8a00d00d857fad40f14fce1d1496fea071e334f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568465 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
-
Michael Lippautz authored
Adds concurrent marking for reaching through v8::TracedReference. Before this CL, a v8::TracedReference would always be processed on the main thread by pushing a callback for each encountered reference. This CL now wires up concurrent handling for such references. In particular: - Global handles are already marked as well and not repurposed during the same GC cycle. - Since global handles are not repurposed, it is enough to double-deref to the V8 object, checking for possible null pointers. - The bitmap for global handle flags is mostly non-atomic, with the markbit being the exception. - Finally, all state is wired up in CppHeap. Concurrent markers keep their own local worklist while the mutator marker directly pushes to the worklist owned by V8. Bug: v8:12600 Change-Id: Ia67dbd18a57dbcccf4dfb9ccfdb9ee438d27fe71 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3516255Reviewed-by:
Omer Katz <omerkatz@chromium.org> Reviewed-by:
Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#79736}
-
Alexander Schulze authored
Bug: v8:12581 Change-Id: I6c665161a3f376321feb773d809c5453ef1d46f9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568445Reviewed-by:
-
Clemens Backes authored
Even if the instructions cannot be dissembled, it still often helps to know which range of instructions belongs to which function. Thus print this information on --print-wasm-code if the disassembler is not available. R=thibaudm@chromium.org Change-Id: I9e7a4cb6ae4edf3411740fe4dfee248a5b9439ec Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3563564Reviewed-by:
Thibaud Michaud <thibaudm@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#79734}
-
Igor Sheludko authored
... instead of Code objects. This is a step towards not creating Code objects for embedded builtins. Bug: v8:11880 Change-Id: Ie9f87b09d06e6b872ce3a5fa5d03a2502df979d9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3564565Reviewed-by:
-
Michael Achenbach authored
No-Try: true Bug: v8:12748 Change-Id: Ide093521c9d1c5927c5c1bea30a6f7689486f079 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3562983 Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Maya Lekova <mslekova@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#79732}
-
Camillo Bruni authored
Change-Id: I38d0f52b7add9247af50aa5f470a88587e97203f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3565725Reviewed-by:
-
Alexander Schulze authored
Bug: v8:12581 Change-Id: I73d8248bd109c906aa0296b3b7bfc19ffd1c00ff Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3568447 Auto-Submit: Alexander Schulze <alexschulze@chromium.org> Reviewed-by:
Tamer Tas <tmrts@chromium.org> Reviewed-by:
-
Jakob Gruber authored
.. since they are the same as eager deopts (% an unused counter). Fixed: v8:12765 Change-Id: I2be6210e476ead4ac6629a49259f28321e965867 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3565717Reviewed-by:
-
