- 25 Mar, 2021 37 commits
-
-
Junliang Yan authored
This is a reland of 0655aa05 Original change's description: > Reland "s390x: [liftoff] implement CallRuntimeStub" > > This is a reland of 88c1ae95 > > Original change's description: > > s390x: [liftoff] implement CallRuntimeStub > > > > Change-Id: I505ea67af31006f3f233eef390f234cfc0188d6c > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2774479 > > Reviewed-by: Milad Fa <mfarazma@redhat.com> > > Commit-Queue: Junliang Yan <junyan@redhat.com> > > Cr-Commit-Position: refs/heads/master@{#73525} > > Change-Id: Id5d0ece78533439870fdc6b000026fe04e576448 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2774561 > Reviewed-by: Milad Fa <mfarazma@redhat.com> > Commit-Queue: Junliang Yan <junyan@redhat.com> > Cr-Commit-Position: refs/heads/master@{#73549} Change-Id: I287a833a3c12869377b83bd66a1bf096f514e034 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787194Reviewed-by: Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/master@{#73689}
-
Ng Zhi An authored
Left i16x8.extmul_low in the arch-specific macro-assemblers because they rely on other functions defined in the same file. We can come back and move it afterwards. Bug: v8:11589 Change-Id: I2ea81c50ed52cc3e59e001b5e80aaf6b93a6572c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786280Reviewed-by: Bill Budge <bbudge@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73688}
-
Ross McIlroy authored
AddImmediate ends up pushing repeated immediates very often unecessarily. Add support for Int64 immediates being inlined into InstructionOperand if they fit into the payload (which is almost always the case). Also add a seperate rpo_immediate vector for RPO numbers to avoid having to add them to the immediates_ vector multiple times. Ideally the RPO values would also be inlined, however JumpThreading needs to patch RPO targets throughout the instruction stream, so we need an indirection. Change-Id: I75b1cdb05917f85d4f740a34c3720dd9cf0ee29c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2782282 Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#73687}
-
Junliang Yan authored
Change-Id: I56b00f00f7b08dee881f473ecedd9596151dee4c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786858Reviewed-by: Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/master@{#73686}
-
Frank Emrich authored
This CL enables proper testing for the CSA/Torque implementation of SwissNameDictionary. This is done by changing the implementation of test_swiss_hash_table::CSATestRunner. This is the last CL of this series and concludes the CSA/Torque implementation of SwissNameDictionary. Bug: v8:11330 Change-Id: Ib46ef3a2c9ec9746591c2af94346be7d0382122b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786843 Commit-Queue: Frank Emrich <emrich@google.com> Reviewed-by: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#73685}
-
Milad Fa authored
Change-Id: I9a1a236185614e41d6dba25331a0731666aed093 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787187 Commit-Queue: Milad Fa <mfarazma@redhat.com> Reviewed-by: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/master@{#73684}
-
Zhi An Ng authored
This reverts commit 9a6567b4. Reason for revert: MSVC compile failures https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Win64%20-%20msvc/b8851762212188482624/steps?succeeded=true&debug=true Original change's description: > [wasm-simd] Move test helpers into wasm-simd-utils > > Move many test helpers into wasm-simd-utils. These helper functions can > potentially be useful for relaxed-simd test in the future. I left behind > simd specific test helpers, like load extend helpers, because those are > for simd instructions. > > Bug: v8:11583,v8:11384 > Change-Id: Id9ed452b06eaf5c97a5dda174b53a37aede2a937 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2783295 > Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73675} Bug: v8:11583 Bug: v8:11384 Change-Id: I88bcefecd62c9b453d0f51ad9b9a912339b110eb No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787239 Auto-Submit: Zhi An Ng <zhin@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/master@{#73683}
-
Ng Zhi An authored
We already skip all tests on no-simd-sse builds, in effect not testing scalar lowering. So, remove explicit tests for scalar lowering for all other build configurations. Cq-Include-Trybots: luci.v8.try:v8_linux_optional_rel_ng Change-Id: I0aedad255236fa029e911bb9dc2dccab419791a4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2728249Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73682}
-
Frank Emrich authored
This CL adapts some existing dictionary-related operations in CSA to support SwissNameDictionary, too. This is needed to make the CSA version of the SwissNameDictionary test suite working. Bug: v8:11330 Change-Id: Ieb315a0806d8910e50f3643abb9a68200485e4fd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784689 Commit-Queue: Frank Emrich <emrich@google.com> Reviewed-by: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#73681}
-
Deepti Gandluri authored
This reverts commit 52393b90. Reason for revert: Reverting for TSAN fails - https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20no-concurrent-marking/3061 Original change's description: > [sparkplug] OSR Ignition -> Sparkplug > > Add support for OSR to baseline code. > We compile baseline and perform OSR immediately when the bytecode budget > interrupt hits. > > Drive-by: Clean-up deoptimizer special handling of JumpLoop by using > the newly introduced GetBaselinePCForNextExecutedBytecode instead of > GetBaselineEndPCForBytecodeOffset. > > Bug: v8:11420 > Change-Id: Ifbea264d4a83a127dd2a11e28626bf2a5e8aca59 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784687 > Commit-Queue: Patrick Thier <pthier@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73677} Bug: v8:11420 Change-Id: I335640216dbbf9a854fc276f3df95bf5a1f9956a No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787192Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#73680}
-
Leszek Swirski authored
This also makes OOMing on --always-sparkplug not crash Bug: v8:11420 Change-Id: Ibaa5eb8a268a7d47ec30210f5cf5af51e514f039 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786846 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73679}
-
Ng Zhi An authored
Bug: v8:11589 Change-Id: I3f1c6d1ece6c634915358f30537c9bbabc0cd3b0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2785818 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#73678}
-
Patrick Thier authored
Add support for OSR to baseline code. We compile baseline and perform OSR immediately when the bytecode budget interrupt hits. Drive-by: Clean-up deoptimizer special handling of JumpLoop by using the newly introduced GetBaselinePCForNextExecutedBytecode instead of GetBaselineEndPCForBytecodeOffset. Bug: v8:11420 Change-Id: Ifbea264d4a83a127dd2a11e28626bf2a5e8aca59 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784687 Commit-Queue: Patrick Thier <pthier@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73677}
-
Ng Zhi An authored
The x64 and ia32 implementations are the same, modulo function signature. x64 has a kScratchDoubleReg available, ia32 takes it as a argument. We standardize on the ia32 function signature and have callers on x64 pass in the scratch register. Bug: v8:11589 Change-Id: I2f75705ed9c618d6e7a7e34ac96b78b772c4944d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786094Reviewed-by: Bill Budge <bbudge@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73676}
-
Ng Zhi An authored
Move many test helpers into wasm-simd-utils. These helper functions can potentially be useful for relaxed-simd test in the future. I left behind simd specific test helpers, like load extend helpers, because those are for simd instructions. Bug: v8:11583,v8:11384 Change-Id: Id9ed452b06eaf5c97a5dda174b53a37aede2a937 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2783295Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73675}
-
Frank Emrich authored
This CL adds: a) FindEntry, Add, and Delete operations on SwissNameDictionary in Torque b) Helpers needed for that on both the CSA and Torque side Bug: v8:11330 Change-Id: I549b9039108217a34ec1e95af84eb28af7dfc9d3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2778175Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Frank Emrich <emrich@google.com> Cr-Commit-Position: refs/heads/master@{#73674}
-
Zhi An Ng authored
This reverts commit 9eba2d85. Reason for revert: TSAN failures https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20isolates/14265/overview Original change's description: > [fastcall] Add fast API testing facilities to d8 > > This CL provides the minimum necessary functionality to expose fast API > for testing in mjsunit, exposing the fast path for fuzzing. It exposes > a d8.test.fast_c_api with an `add_all` method, which exercises primitive > types. On x64, all integer and floating point types are supported. On > other platforms currently only 32-bit integers are included in the test. > > Design doc: > https://docs.google.com/document/d/1KUKPfXkSRZTA2gMwaWbpQKlYfw0C-T6AE3XzC4viHbo/ > > Bug: chromium:1052746 > Change-Id: Icc824199a26dd2abd2b869f5483a39d38e4dce3e > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2749154 > Reviewed-by: Camillo Bruni <cbruni@chromium.org> > Reviewed-by: Georg Neis <neis@chromium.org> > Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> > Commit-Queue: Maya Lekova <mslekova@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73670} Bug: chromium:1052746 Change-Id: Iaf5083540ddfe882a747eaa9d1d2a2a8b4ba0ec0 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2787081 Auto-Submit: Zhi An Ng <zhin@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/master@{#73673}
-
Igor Sheludko authored
... and don't enable them for CFI builds. Bug: v8:11527 Change-Id: I651473833c1e74d942bd75dc7574dbca45b34b64 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786855Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#73672}
-
Andreas Haas authored
When copying the setup of the WebAssembly.Exception constructor to InstallConditionalFeatures, I forgot to copy some parts. R=thibaudm@chromium.org Bug: chromium:1191853 Change-Id: Iacba30b3356fc42b7d97f8c105ef91b776b95eb4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786853Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#73671}
-
Maya Lekova authored
This CL provides the minimum necessary functionality to expose fast API for testing in mjsunit, exposing the fast path for fuzzing. It exposes a d8.test.fast_c_api with an `add_all` method, which exercises primitive types. On x64, all integer and floating point types are supported. On other platforms currently only 32-bit integers are included in the test. Design doc: https://docs.google.com/document/d/1KUKPfXkSRZTA2gMwaWbpQKlYfw0C-T6AE3XzC4viHbo/ Bug: chromium:1052746 Change-Id: Icc824199a26dd2abd2b869f5483a39d38e4dce3e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2749154Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#73670}
-
Z Nguyen-Huu authored
Change-Id: Iea2fa346bb4d6cd34a1e3f892b9787e75f26237d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786681Reviewed-by: Seth Brenith <seth.brenith@microsoft.com> Commit-Queue: Z Nguyen-Huu <duongn@microsoft.com> Cr-Commit-Position: refs/heads/master@{#73669}
-
Leszek Swirski authored
Disable or support --always-sparkplug in various tests that until now were always expecting only bytecode. Bug: v8:11420 Change-Id: Ida45041739fb55851aa493c51f1ed796aa1c0606 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786852Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73668}
-
Leszek Swirski authored
Fix a broken DCHECK when logging existing baseline code. Bug: v8:11420 Change-Id: I66cf1b1b806aef74bcc9c1a0061bec1834b719f6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786850Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73667}
-
Leszek Swirski authored
CreateShallowArrayLiteral can fail and bail out to the runtime. We therefore have to pass the literal flags to the builtin, so that the runtime e.g. initialises the AllocationSite correctly. Bug: v8:11420 Change-Id: I57dfbbc202aa2c3b0e7ac01f65ee84f6e3763180 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786848Reviewed-by: Maya Lekova <mslekova@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73666}
-
Leszek Swirski authored
JSFunctions can store a pointer to baseline code, which would trigger that code to be serialized in the context serializer. Since we don't support baseline code serialization, we have to flush these baseline code pointers. This is similar to the existing optimized code flushing. Bug: v8:11420 Change-Id: Idabd0caea24aed484d5cf39ad0dad8c2c8180b91 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786849Reviewed-by: Patrick Thier <pthier@chromium.org> Commit-Queue: Patrick Thier <pthier@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73665}
-
Leszek Swirski authored
Some baseline implementations of non-accumulator-writing bytecodes would call builtins without saving the accumulator. This clobbers the accumulator. This was particularly observable with IncBlockCounter. Bug: v8:11420 Change-Id: I1e8a06526ea6ed018da0d6213775cce130c72aee Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786847 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/master@{#73664}
-
Leszek Swirski authored
Use a local context and compilation cache clearing to trigger code deletion in ClearUnusedWithEagerLogging, rather than relying on bytecode flushing. This allows the test to succeed with non-flushable native-context-independent code, like Sparkplug baseline code. Bug: v8:11420 Change-Id: Id6e3626b6245b4308d95aa3995a8fb2ee53684f7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786851Reviewed-by: Yang Guo <yangguo@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#73663}
-
Milad Fa authored
We will need to use both the vector multiply even and odd instructions. Change-Id: I1af3517dc3359d395682e3d661068b429edba8f1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2785427Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#73662}
-
Milad Fa authored
This reverts commit d8c6b2ec. Reason for revert: original cl has relanded Original change's description: > Revert "PPC/s390: [sparkplug][deoptimizer] Deoptimize to baseline." > > This reverts commit 94272ea5. > > Reason for revert: original port was reverted > > Original change's description: > > PPC/s390: [sparkplug][deoptimizer] Deoptimize to baseline. > > > > Port bdcd7d79 > > > > Original Commit Message: > > > > If we have baseline code, deoptimize to baseline instead of the > > interpreter. The process is similar to deopting to the interpreter. > > We just use different builtins > > (BaselineEnterAtBytecode/BaselineEnterAtNextBytecode) instead of > > InterpreterEnterBytecodeDispatch/InterpreterEnterBytecodeAdvance, that > > patch an interpreter frame to a baseline frame and continue execution in > > baseline code (based on the deopt type, at the current or next > > bytecode). > > > > R=pthier@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com > > BUG= > > LOG=N > > > > Change-Id: I3230f3f3c6506230b2751a3389f10b022dec61a3 > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2783022 > > Reviewed-by: Junliang Yan <junyan@redhat.com> > > Commit-Queue: Milad Fa <mfarazma@redhat.com> > > Cr-Commit-Position: refs/heads/master@{#73618} > > Change-Id: I903ad90099c4dc5f153d28aea9246933ac69972b > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784002 > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Commit-Queue: Milad Fa <mfarazma@redhat.com> > Cr-Commit-Position: refs/heads/master@{#73630} Change-Id: I7746f42472ae63e49770f491ae6b8ef7b596cfce Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2785380 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#73661}
-
Nico Hartmann authored
The implementation of Flatten() Torque generates for CSA struct types and the current implementation of EmitCCValue make heavy use of std::make_tuple and std::tuple_cat, which had a noticeable impact on build times of generated files including these outputs. This CL eliminates all uses of std::tuple_cat and most uses of std::make_tuple. Bug: v8:9861 Change-Id: Ib950b2c87983171b8c32c388fc115d2407087717 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784563 Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Reviewed-by: Seth Brenith <seth.brenith@microsoft.com> Cr-Commit-Position: refs/heads/master@{#73660}
-
Michael Achenbach authored
This reverts commit db16dce2. Reason for revert: https://ci.chromium.org/p/v8/builders/ci/V8%20Blink%20Linux%20Debug/8771 Original change's description: > [api] Assign serial numbers when template infos are added to cache > > Instead of assigning serial numbers when the template infos are > created, this patch creates serial numbers only when they are added to > cache. > > This way only the ones that are first instantiated are allocated the > fast template cache. Previously, various accessors and methods that > would almost never get instantiated got assigned to the fast template > cache. > > Bug: v8:11284 > Change-Id: I6b633e56e59cbfc3fa5d4ee2db53ca2849eecdd7 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2621081 > Reviewed-by: Camillo Bruni <cbruni@chromium.org> > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73655} Bug: v8:11284 Change-Id: I382915b2c1be1d87d7a7a961d13e1dd5e3951a4f No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2786844 Auto-Submit: Michael Achenbach <machenbach@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/master@{#73659}
-
Andreas Haas authored
The installation of conditional features in WebAssembly needs the context to be set in the isolate to fix https://ci.chromium.org/ui/p/chromium/builders/try/linux-rel/b8851955177753045904/overview. The context cannot be passed explicitly to all uses of the context, because Isolate::object_function() accesses the native_context through the isolate. Change-Id: I20334fd6779326e64825a15ba2aca3ae06a5c0d7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784684Reviewed-by: Camillo Bruni <cbruni@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#73658}
-
Sam Parker authored
Add Float32Select and Float64Select as OptionalOperators and insert these, if supported, when handling a Select expression in the wasm graph builder. FlagsContinuation have been modified to support the select operation and code generation support has been added for arm64. This improves the 'Bullet' physics benchmark by ~2-3%. Change-Id: I928c3085c9136ad8baeeb34c71c47c1c8338844c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2763871 Commit-Queue: Martyn Capewell <martyn.capewell@arm.com> Reviewed-by: Andreas Haas <ahaas@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#73657}
-
Patrick Thier authored
This is a reland of e3ccb538 No changes for the reland. This CL was speculatively reverted, but was not the cause of the problem. TBR=jgruber@chromium.org Original change's description: > Reland "[sparkplug][deoptimizer] Deoptimize to baseline." > > This is a reland of bdcd7d79 > > Handle lazy deopts when the current bytecode is JumpLoop. > Instead of advancing to the next bytecode, re-execute the JumpLoop. > > TBR=jgruber@chromium.org, neis@chromium.org > > Original change's description: > > [sparkplug][deoptimizer] Deoptimize to baseline. > > > > If we have baseline code, deoptimize to baseline instead of the > > interpreter. The process is similar to deopting to the interpreter. > > We just use different builtins > > (BaselineEnterAtBytecode/BaselineEnterAtNextBytecode) instead of > > InterpreterEnterBytecodeDispatch/InterpreterEnterBytecodeAdvance, that > > patch an interpreter frame to a baseline frame and continue execution in > > baseline code (based on the deopt type, at the current or next > > bytecode). > > > > Bug: v8:11420 > > Change-Id: Iabaefb36c05155a435c7b380906a86d9b9d549fa > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2695591 > > Commit-Queue: Patrick Thier <pthier@chromium.org> > > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > > Reviewed-by: Georg Neis <neis@chromium.org> > > Reviewed-by: Jakob Gruber <jgruber@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#73609} > > Bug: v8:11420 > Change-Id: Ib8cac028121188ddc23ff29377760ed684eb7392 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2783035 > Reviewed-by: Patrick Thier <pthier@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Commit-Queue: Patrick Thier <pthier@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73636} Bug: v8:11420 Change-Id: I7fbbb73a4fdaeab8b294862ee6ae952928c57994 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784695 Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Patrick Thier <pthier@chromium.org> Cr-Commit-Position: refs/heads/master@{#73656}
-
Sathya Gunasekaran authored
Instead of assigning serial numbers when the template infos are created, this patch creates serial numbers only when they are added to cache. This way only the ones that are first instantiated are allocated the fast template cache. Previously, various accessors and methods that would almost never get instantiated got assigned to the fast template cache. Bug: v8:11284 Change-Id: I6b633e56e59cbfc3fa5d4ee2db53ca2849eecdd7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2621081Reviewed-by: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#73655}
-
Sathya Gunasekaran authored
This reverts commit c10aa432. Reason for revert: https://ci.chromium.org/ui/p/chromium/builders/try/mac-rel/b8851806981073807856/overview Original change's description: > [sparkplug] Enable short builtin calls by default > > ... when they are enabled via the GN flag. > > Also disable this optimizaion for CFI builds for now. > > Sheriffs: This CL adds ~1.2-1.4Mb of memory overhead per Isolate on > certain configurations and that's expected. This is a temporary fix > until the full solution is implemented. > See the design doc linked to the issue for details. > > Bug: v8:11527 > Change-Id: I7444c6878ea58d67ef580cf50cb550b3820c85c4 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784690 > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Cr-Commit-Position: refs/heads/master@{#73650} Bug: v8:11527 Change-Id: I96ff92e3f5c1a1a7ebe68d7f9d2981b5cf9c0094 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784697Reviewed-by: Sathya Gunasekaran <gsathya@chromium.org> Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#73654}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/f628144..7b935eb Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/731dd85..f50df92 Rolling v8/third_party/aemu-linux-x64: oZxl99tyPs7o9Eq0hlPel1m4iyPu1Z92wj2Llb6HWwEC..wvBTICmOxqrOyqTGFXVMcKyHB3KmKkB4Xo6VVahwXs8C Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/dbc2c42..4f773bc TBR=v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I11a315567cd12d4010a191f4b6e5fad8f9735c82 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2785805Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#73653}
-
- 24 Mar, 2021 3 commits
-
-
Ng Zhi An authored
SharedTurboAssembler is shared between ia32 and x64. It contains shared functionality between these two backends. It derives from TurboAssemblerBase, and the backend-specific TurboAssembler derives from SharedTurboAssembler. That way, LiftoffAssembler does not need to change. As an example, we move a single member function I64x2SConvertI32x4High into SharedTurboAssembler. This function has the exact same implementation on ia32 and x64, and is now defined in a single place. Bug: v8:11589 Change-Id: I1c4a0628d61960e7398d458fb813561751e0614f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2773609Reviewed-by: Toon Verwaest <verwaest@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#73652}
-
Frank Emrich authored
This CL adds: a) Allocation and Copy operations on the CSA side b) Helpers needed for that on both the CSA and Torque side Bug: v8:11330 Change-Id: I322c84e1762d2d3503a0d4809c6e1c0bc9d442e1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2775563Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#73651}
-
Igor Sheludko authored
... when they are enabled via the GN flag. Also disable this optimizaion for CFI builds for now. Sheriffs: This CL adds ~1.2-1.4Mb of memory overhead per Isolate on certain configurations and that's expected. This is a temporary fix until the full solution is implemented. See the design doc linked to the issue for details. Bug: v8:11527 Change-Id: I7444c6878ea58d67ef580cf50cb550b3820c85c4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2784690 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#73650}
-