- 25 Jun, 2021 3 commits
-
-
Clemens Backes authored
We were just printing "Liftoff" in the compiler field, whichout a way to tell if it's Liftoff's debugging code flavour or default Liftoff code. This CL encodes that in the "compiler" field by selecting between "TurboFan", "Liftoff", and "Liftoff (debug)". R=ahaas@chromium.org Change-Id: Ia0c004eab96c0e10e52e81c4e2863dab423124fe Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2985399Reviewed-by: Andreas Haas <ahaas@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#75377}
-
John Xu authored
These changes are found after Cobalt rebased to V8 v8.8. Most changes are grouped into their own CLs and here are the rest. - Replace raw printf with PrintF - Include Starboard as a platform for some implementations - Add some missing headers. This is probably because Cobalt does not compile all of V8 sources so some missing headers are only exposed when compiling Cobalt. Change-Id: Idf1512f57c37c8a63f79b1cf4ffc1da5d08421e2 Bug: v8:10927 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2966197 Auto-Submit: John Xu <johnx@google.com> Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: John Xu <johnx@google.com> Cr-Commit-Position: refs/heads/master@{#75376}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/62937f9..1799e1b prefix VLOG (#1187) (Dominic Hamon) https://chromium.googlesource.com/external/github.com/google/benchmark/+/1799e1b prefix macros to avoid clashes (#1186) (Dominic Hamon) https://chromium.googlesource.com/external/github.com/google/benchmark/+/6a5bf08 Move flags inside the `benchmark` namespace (#1185) (Dominic Hamon) https://chromium.googlesource.com/external/github.com/google/benchmark/+/5da5660 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: I4e2e782f3ac6d80c2aaacecb3b9825db2d44cef9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2987700Reviewed-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@{#75375}
-
- 24 Jun, 2021 32 commits
-
-
Victor Gomes authored
No-Try: true Bug: v8:11234 Change-Id: I4099a231ecc5cfb87d74170b04753a8bec2b5cbf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2985501Reviewed-by: Dan Elphick <delphick@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Auto-Submit: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/master@{#75374}
-
Peter Kasting authored
These indicate when a range-based for loop is using an index whose type (value, pointer, or reference) doesn't match what the loop actually extracts from the range. Fix by matching the actual type better. This shouldn't cause any behavior/performance change, just be slightly clearer about what's actually happening when reading the code. Bug: chromium:1223264 Change-Id: Ib8773fbbeb038609c54a52c7cd6ce5bd11fd99ba Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983710Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#75373}
-
Victor Gomes authored
- Disables FRIEND_TEST - Removes gtest_prod.h dependency Change-Id: I614f633749f3197989ac4a0d9d5faa6ec03ed59d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982481Reviewed-by: Igor Sheludko <ishell@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/master@{#75372}
-
Victor Gomes authored
No-Try: true Bug: v8:11234 Change-Id: I4339a3a3f3fcffa7c65a569ea36a56be16b1ae45 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2985340 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#75371}
-
Victor Gomes authored
Bug: v8:11234 No-Try: true Change-Id: Idc8a0d6a513fd38fafb10ff33121e37a79e3a1f3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2985242Reviewed-by: Dan Elphick <delphick@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Auto-Submit: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/master@{#75370}
-
Maya Lekova authored
This CL modifies the underlying storage of PerIsolateAssertScope from a bitfield to separate booleans. This slightly increases the space taken by the isolate, but allows for easier access to the individual fields, which is a prerequisite for implementing assertion scopes in TurboFan. It also refactors the template PerIsolateAssertScope class to separate simple C++ scope classes, defined through macros. Bug: chromium:1218898 Change-Id: Ia5e43352ebba28be6f013376b75f13ec8d5dc972 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2975303 Commit-Queue: Maya Lekova <mslekova@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#75369}
-
Dan Elphick authored
This is a reland of 9701d4a4 with a small fix for some code landed in between the dry-run and submission. Original change's description: > [base] Move most of src/numbers into base > > Moves all but conversions.*, hash-seed-inl.h and math-random.* into > base, in preparation for moving the parts of conversions that don't > access HeapObjects. > > Also moves uc16 and uc32 out of commons/globals.h into base/strings.h. > > Bug: v8:11917 > Change-Id: Ife359148bb0961a63833aff40d26331454b6afb6 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2979595 > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> > Reviewed-by: Clemens Backes <clemensb@chromium.org> > Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> > Auto-Submit: Dan Elphick <delphick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#75354} Bug: v8:11917 Change-Id: Ie1ec9032fe56646a7c7303185cecc70fce5694ae Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982607Reviewed-by: Clemens Backes <clemensb@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#75368}
-
Jakob Kummerow authored
Instead, make the array-allocating builtin initialize the object. This speeds up later stages of Turbofan graph processing, in particular live range computation. Bug: v8:7748 Change-Id: Iba0d682922b444b1d6151eeaee8d939821ebc980 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983457 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Reviewed-by: Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/master@{#75367}
-
Clemens Backes authored
There is only one global wasm engine, so we do not need to store the pointer in the NativeModule. We just use {GetWasmEngine()} instead, which reads the global pointer. R=jkummerow@chromium.org Bug: v8:11879 Change-Id: I66dedd571755774d96621b8d20ff23bdfef8134f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983208Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#75366}
-
Victor Gomes authored
Supports cross-compilation targeting android x86: bazel build --config=android --android_cpu=x86 --cpu=x86 \ --host_cpu=k8 :d8 Caveat: one needs to have a cross compiler setup. No-Try: true Bug: v8:11234 Change-Id: Ieff72d9c04c8c3db9801c6316bae5c2570265f45 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983211Reviewed-by: Dan Elphick <delphick@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/master@{#75365}
-
Jakob Kummerow authored
We've seen Turbofan take excessive amounts of time for very large functions. As a short-term mitigation, this patch disables optimization for functions larger than 500KB (in their Wasm binary format). Functions of such sizes are rare, so most modules should be unaffected by this patch. Change-Id: I9d222df5ca51b0fb5d6db7a7e9e3402f5276ff38 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982608Reviewed-by: Andreas Haas <ahaas@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/master@{#75364}
-
Nico Hartmann authored
Bug: v8:11922 Change-Id: I9836b9ab2b8667b2c08f9525a386566117d95598 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982606 Auto-Submit: Nico Hartmann <nicohartmann@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#75363}
-
Nico Hartmann authored
This reverts commit 8f17052c. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20stress-incremental-marking/3735/overview Original change's description: > [test] Unmark TSAN tests as SLOW/SKIP > > We allowed more time for tsan testing [1] so unmarking some tests. > > [1]: https://chromium-review.googlesource.com/c/v8/v8/+/2979671 > > Bug: v8:7790, v8:11600, v8:11906 > Change-Id: I026a32abe0380c2d32327aaa8b301ef51f22a5e2 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2979602 > Reviewed-by: Mythri Alle <mythria@chromium.org> > Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> > Cr-Commit-Position: refs/heads/master@{#75357} Bug: v8:7790, v8:11600, v8:11906 Change-Id: Ib582d9726aa7e7fe4984549f5b344ff8413cc0c4 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983210Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/master@{#75362}
-
Maya Lekova authored
This CL removes an unnecessary CHECK and replaces it with exception throwing instead. Bug: chromium:1223107 Change-Id: If99bf289fe3a2ed5cd675ee071e9b39038e38129 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983201 Auto-Submit: Maya Lekova <mslekova@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#75361}
-
Santiago Aboy Solanes authored
Bug: v8:7790 Change-Id: I5e38137b5c01119e47c98879fe549b629343e7ce Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982341 Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by: Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#75360}
-
Victor Gomes authored
No-Try: true Change-Id: Id8c7aa494bda905aef9cc605e206df685de30233 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983534 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Dan Elphick <delphick@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#75359}
-
Santiago Aboy Solanes authored
Bug: v8:7790 Change-Id: I7e741b659daa41b4a3144cc4170fff4f488968ba Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982340Reviewed-by: Georg Neis <neis@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#75358}
-
Santiago Aboy Solanes authored
We allowed more time for tsan testing [1] so unmarking some tests. [1]: https://chromium-review.googlesource.com/c/v8/v8/+/2979671 Bug: v8:7790, v8:11600, v8:11906 Change-Id: I026a32abe0380c2d32327aaa8b301ef51f22a5e2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2979602Reviewed-by: Mythri Alle <mythria@chromium.org> Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Cr-Commit-Position: refs/heads/master@{#75357}
-
Nico Hartmann authored
This reverts commit 9701d4a4. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Mac64/40802/overview Original change's description: > [base] Move most of src/numbers into base > > Moves all but conversions.*, hash-seed-inl.h and math-random.* into > base, in preparation for moving the parts of conversions that don't > access HeapObjects. > > Also moves uc16 and uc32 out of commons/globals.h into base/strings.h. > > Bug: v8:11917 > Change-Id: Ife359148bb0961a63833aff40d26331454b6afb6 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2979595 > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> > Reviewed-by: Clemens Backes <clemensb@chromium.org> > Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> > Auto-Submit: Dan Elphick <delphick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#75354} Bug: v8:11917 Change-Id: Iacf796c95256016fa74f0a910c5bb1a86baa425a No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982605 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/master@{#75356}
-
Thibaud Michaud authored
Because of catch-less try blocks, we have to check the exception stack one more time so that we don't try to rethrow to an outer try scope if this is already the outermost one. R=clemensb@chromium.org Bug: chromium:1219746 Change-Id: I6ad7e4afec669f04bc10197998db5d6eb38a2205 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983205Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#75355}
-
Dan Elphick authored
Moves all but conversions.*, hash-seed-inl.h and math-random.* into base, in preparation for moving the parts of conversions that don't access HeapObjects. Also moves uc16 and uc32 out of commons/globals.h into base/strings.h. Bug: v8:11917 Change-Id: Ife359148bb0961a63833aff40d26331454b6afb6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2979595Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Auto-Submit: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#75354}
-
Mathias Bynens authored
The lists are to be sorted line by line, which is easier to do than sorting based on an arbitrarily-positioned substring within each line. Change-Id: Ie5e6552e2468545a5800c4bc145cc2b4da1ed4a0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982601 Auto-Submit: Mathias Bynens <mathias@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#75353}
-
Ross McIlroy authored
Adds support for Turboprop, limiting the amount of bytecode that can be inlined into a function compared with TurboFan using a scaling factor, currently set to 1/4. To enable inlining, we make sure to only emit the tierup check for the main (non-inlined) function and skip it for any inlined functions. BUG=v8:9684 Change-Id: I1399e2b275f797a5d3fd49c89f23296f75439eee Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982337Reviewed-by: Santiago Aboy Solanes <solanes@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#75352}
-
Camillo Bruni authored
This is a reland of 2588380d Fixes arm compilation. Original change's description: > [assembler][arm64][arm][ia32] Add more Assembler::CodeComment > > Use more code comments to better document generated assembler code. > This CL also attaches a CodeComment to FrameScope. > > Change-Id: I45afc3e4078dc161316b877fbbd1ae8bcd6e821d > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2964392 > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> > Reviewed-by: Victor Gomes <victorgomes@chromium.org> > Commit-Queue: Camillo Bruni <cbruni@chromium.org> > Cr-Commit-Position: refs/heads/master@{#75277} Change-Id: I150c9166e0d31e4af857f76c567b829d16da87e4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2976651Reviewed-by: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#75351}
-
Liu Yu authored
Port: e699762e Bug: v8:11856 Change-Id: Ib832dfcee864b21b4e56498fca54e9ae01c44ae7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983711 Auto-Submit: Liu yu <liuyu@loongson.cn> Reviewed-by: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Cr-Commit-Position: refs/heads/master@{#75350}
-
Sigurd Schneider authored
Bug: chromium:1213393 Change-Id: I56817237424bb83aa9e5d06a47187f245c3cae0e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983200 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#75349}
-
Timothy Gu authored
Change-Id: I3ec23a8969e311be3afe684df4a545bec0453449 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2971404Reviewed-by: Hannes Payer <hpayer@chromium.org> Commit-Queue: Hannes Payer <hpayer@chromium.org> Auto-Submit: Timothy Gu <timothygu@chromium.org> Cr-Commit-Position: refs/heads/master@{#75348}
-
Lei Zhang authored
Change-Id: I9dbca09523644934eefb14fdb60f360b28417fa2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983712Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#75347}
-
Mathias Bynens authored
Bug: chromium:1213438 Change-Id: I52e381841b888e0f5edc0bad9e0319e840829914 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982600 Commit-Queue: Mathias Bynens <mathias@chromium.org> Commit-Queue: Hannes Payer <hpayer@chromium.org> Auto-Submit: Mathias Bynens <mathias@chromium.org> Reviewed-by: Hannes Payer <hpayer@chromium.org> Cr-Commit-Position: refs/heads/master@{#75346}
-
Patrick Thier authored
- Increase initial queue capacity from 4 to 32 to reduce growing/copying of the queue. - In addition remove the HandleScopes to reduce overhead. The only caller (BytecodeBudgetInterruptFromBytecode) already creates one. Bug: v8:11790 Change-Id: I33cb9222706f9b349c56a7ea14142a930b8ecbf8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982012Reviewed-by: Toon Verwaest <verwaest@chromium.org> Commit-Queue: Patrick Thier <pthier@chromium.org> Cr-Commit-Position: refs/heads/master@{#75345}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/e589a83..9dc1297 Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/9c143ac..46bcd59 Rolling v8/buildtools/linux64: git_revision:e9b8433248ae2c117644b4e40b33203e7d3da192..git_revision:d924640c25f9d90386716116a53957f24d709042 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/f4328ad..bf260ac Rolling v8/third_party/aemu-linux-x64: LieFB6bFq-7RUGMVRhbOR3IXo6WdPE7fCzCMv4PnaScC..JZYvEBpGNCDNOK3uLHlpKn_AN6roNsL0YVup96egRSEC Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/09f358b..cf60c0a Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/8dd9dda..098695c TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I29cdcfc4ef39ce45c615634b1b1452d776af5604 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983847Reviewed-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@{#75344}
-
Zheng Liu authored
Check whether the right most 16 bits are all-zero. Change-Id: I13bb8856888cbabc19c1f9354048f05ff9e4aacb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983839Reviewed-by: Ji Qiu <qiuji@iscas.ac.cn> Commit-Queue: Ji Qiu <qiuji@iscas.ac.cn> Cr-Commit-Position: refs/heads/master@{#75343}
-
- 23 Jun, 2021 5 commits
-
-
Milad Fa authored
Port e699762e Original Commit Message: Instrument floating-point operations to set a flag if the result is NaN. Does not handle f32x4 and f64x2 results yet. R=thibaudm@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: If81861b65d2a0a98389eebb480127069fd1b5509 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983458Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#75342}
-
Milad Fa authored
The immediate offset passed to `lay` might be bigger than the allowed 20 bits, in which case we need to add it to the base register manually. Issue came up after this CL https://crrev.com/c/2904926 when a related test case started failing as the offset was changed. Change-Id: I9997bf0ed4e42e32ac52bbd2771bbcb13a328e64 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2983164Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#75341}
-
Timothy Gu authored
https://github.com/tc39/ecma262/pull/1490 changed the spec so that the "name" property of a class should be installed after "length" but before "prototype". This CL adapts accordingly. After this change, there is now no need for the separate code path to set the "name" accessor at runtime. Delete the relevant runtime code as well. Bug: v8:8771 Change-Id: I8f809b45bf209c899cf5df76d0ebf6d9a45a6d4e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2974772 Commit-Queue: Timothy Gu <timothygu@chromium.org> Reviewed-by: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#75340}
-
Adam Kallai authored
SNPrintF function is moved into base/strings.h [1], so it needs to fix the scope of this function for Windows on ARM related source as well. [1] https://chromium-review.googlesource.com/c/v8/v8/+/2972732 Bug: None Change-Id: Ia9934f17941558b6338f28900f069766507c87b2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982016Reviewed-by: Dan Elphick <delphick@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#75339}
-
Clemens Backes authored
This is a reland of ac654646. Two constants defined in {AssemblerBase} were not defined anywhere, which is fixed now. Original change's description: > [wasm] Remove WasmInstructionBuffer > > {WasmInstructionBuffer} was basically a wrapper around {AssemblerBuffer} > which remembered the last {AssemblerBuffer} on {Grow()}. Since the > {Assembler} itself already keeps track of the latest {AssemblerBuffer}, > this functionality is mostly redundant. All we need instead is a method > to retrieve the {AssemblerBuffer} from the {Assembler}. > > This CL thus removes {WasmInstructionBuffer} and instead adds > {AssemblerBase::ReleaseBuffer}. > > R=jkummerow@chromium.org, mslekova@chromium.org > CC=dlehmann@google.com > > Bug: v8:11714 > Change-Id: Id07945b67992802a6177bf09e5f5c5be08f657b0 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982013 > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Reviewed-by: Maya Lekova <mslekova@chromium.org> > Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> > Cr-Commit-Position: refs/heads/master@{#75336} Bug: v8:11714 Change-Id: I8797de1a7a78a93aaef936e46bfd1e73ec2cc9d5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2982015Reviewed-by: Maya Lekova <mslekova@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#75338}
-