- 08 Sep, 2021 7 commits
-
-
Leszek Swirski authored
Change-Id: Ic917bf0e6a3d5f6624f77c72cb9d16626d6508fa Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3145418 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Patrick Thier <pthier@chromium.org> Reviewed-by:
Patrick Thier <pthier@chromium.org> Cr-Commit-Position: refs/heads/main@{#76717}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/4f80705..c23a001 Remove unused parameter from lambda. (#1223) (Dominic Hamon) https://chromium.googlesource.com/external/github.com/google/benchmark/+/c23a001 add to final releasing step (Dominic Hamon) https://chromium.googlesource.com/external/github.com/google/benchmark/+/92f50a2 bump version to 1.6 in preparation for release (Dominic Hamon) https://chromium.googlesource.com/external/github.com/google/benchmark/+/f91b6b4 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: I69c037d549e7d2887962e6d7cfa3ba87d990fe2b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3146793Reviewed-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/main@{#76716}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/03a089b..fbef918 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/d579cbc..d69b31c Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/749fd8e..195c102 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I08604eaa678dda32dda55016fcbb2a71183d1387 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3145268Reviewed-by:
-
Liu Yu authored
I'm one of the v8 maintainers of mips, and I request to add myself to MIPS_OWNERS. Change-Id: If246f44e54497619d9d58929131e3d2d1dfc9c5b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141031 Auto-Submit: Liu yu <liuyu@loongson.cn> Reviewed-by:
Lutz Vahl <vahl@chromium.org> Reviewed-by:
Adam Klein <adamk@chromium.org> Commit-Queue: Liu yu <liuyu@loongson.cn> Cr-Commit-Position: refs/heads/main@{#76714}
-
Ng Zhi An authored
Bug: v8:11879 Change-Id: I02cfb6ca7cff418dc3e4ab422a1bc3437f0ea778 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3146075Reviewed-by:
Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/main@{#76713}
-
Ng Zhi An authored
This is a reland of 65515ddd Fix is to use AddWithWraparound for signed additions to avoid UB. Original change's description: > [wasm-simd][arm64] Fuse add and extmul > > We can select a better instruction for add+extmul, using one of the > multiply-long-accumulate instruction. > > Define a helper struct to pattern match Add(x, OP(y, z)) and > Add(OP(x, y) z), and ensure that the matched OP is always on the > LHS, to simplify checking for matches. > > Bug: v8:11548 > Change-Id: I7ab488b262aa9f749785f973549ccd9fad72f4c8 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2826725 > Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Cr-Commit-Position: refs/heads/main@{#76708} Bug: v8:11548 Change-Id: I675ab8b78d9c6c30b82a8c96c8e7098a548c6a60 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144379 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
-
Andrew Brown authored
A previous change (see ref) added a subset of 256-bit instructions to the x64 assembler--this change adds a disassembly test for the added instructions. ref: https://chromium-review.googlesource.com/c/v8/v8/+/3123648 Change-Id: Ia56be7a7df636b8bf6c04f044912e914d949d19f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3133956 Auto-Submit: Andrew Brown <andrew.brown@intel.com> Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/main@{#76711}
-
- 07 Sep, 2021 30 commits
-
-
Frank Tang authored
https://chromium.googlesource.com/external/github.com/tc39/test262/+log/45a913c0..50f3fca Bug: v8:7834, v8:12167, v8:12168, v8:11544 Change-Id: I1cebc0a82cced63fbf9ec46111825f22ac01eeeb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3142539Reviewed-by:
Shu-yu Guo <syg@chromium.org> Commit-Queue: Frank Tang <ftang@chromium.org> Cr-Commit-Position: refs/heads/main@{#76710}
-
Zhi An Ng authored
This reverts commit 65515ddd. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20UBSan/18117/overview Original change's description: > [wasm-simd][arm64] Fuse add and extmul > > We can select a better instruction for add+extmul, using one of the > multiply-long-accumulate instruction. > > Define a helper struct to pattern match Add(x, OP(y, z)) and > Add(OP(x, y) z), and ensure that the matched OP is always on the > LHS, to simplify checking for matches. > > Bug: v8:11548 > Change-Id: I7ab488b262aa9f749785f973549ccd9fad72f4c8 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2826725 > Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Cr-Commit-Position: refs/heads/main@{#76708} Bug: v8:11548 Change-Id: Ic1560616e7ee6df917fcedbb6ad139a1a9773d68 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144377 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/main@{#76709}
-
Ng Zhi An authored
We can select a better instruction for add+extmul, using one of the multiply-long-accumulate instruction. Define a helper struct to pattern match Add(x, OP(y, z)) and Add(OP(x, y) z), and ensure that the matched OP is always on the LHS, to simplify checking for matches. Bug: v8:11548 Change-Id: I7ab488b262aa9f749785f973549ccd9fad72f4c8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2826725Reviewed-by:
-
Jakob Kummerow authored
On MacOS, `ps -e | grep foo` always finds a match, because `ps` already sees the `grep` command, which matches the search string. Luckily, `pgrep` exists as an elegant alternative. No-Try: true Change-Id: Ieedfe62ab11a59fb9a29ae1ebd39b164eec5193c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144917 Auto-Submit: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#76707}
-
Ng Zhi An authored
Bug: v8:12094 Change-Id: Ibefce881cbfcd4445485197a4a2615bdf0599ada Fixed: v8:12094 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3123638 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
-
Milad Fa authored
Change-Id: Iefd76cdf793578acf46717d99c8896c7df7e1970 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3145179Reviewed-by:
Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#76705}
-
Junliang Yan authored
Change-Id: Id37868f94a9ccebdee0530e9eb450c4acfa2f7c8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3145414Reviewed-by:
Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/main@{#76704}
-
Jakob Kummerow authored
Simply putting all features behind --experimental-wasm-gc. The intent is to simplify command lines. Bug: v8:7748 Change-Id: Ibfaa4dc720087a490b177a2b95841620a4d25d89 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141583 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Auto-Submit: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/main@{#76703}
-
Deepti Gandluri authored
Change-Id: Ie614bf368d6cf955cb1a7537ef81aafd6925ccbb No-Tree-Checks: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144149 Auto-Submit: Deepti Gandluri <gdeepti@chromium.org> Reviewed-by:
-
Toon Verwaest authored
That makes it easier to try various values. Change-Id: I3f4784d148cd5c7524773972e72e1a37ce861210 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2972731 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#76701}
-
Lu Yahan authored
- Add vsetivli/I8x16Add/vl/vse8 - In Rvv, Vector regs is different from Float Regs. But in this cl, in order to facilitate modification, it is assumed that the vector register and float register share a set of register codes. - Because v0 is mask reg, we can't allocate it . And transfer float into vector reg, so i delete ft0 from AllocateReg. Bug: v8:11976 Change-Id: I66185d1f5ead985489bcbdf671b131f02a6bd7c2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3005768 Commit-Queue: Ji Qiu <qiuji@iscas.ac.cn> Reviewed-by:
Thibaud Michaud <thibaudm@chromium.org> Reviewed-by:
Ji Qiu <qiuji@iscas.ac.cn> Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#76700}
-
Patrick Thier authored
The receiver is now always included in the actual argument count and the formal parameter count. kDontAdaptArgumentsSentinel is changed from UINT16_MAX to 0 to preserve the maximum allowed declared parameters. The build flag activating the changes is not set for any architecture yet. Bug: v8:11112 Change-Id: I48a4969137949a1b4d1f47545209bb22b64e7e05 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140608 Commit-Queue: Patrick Thier <pthier@chromium.org> Reviewed-by:
Omer Katz <omerkatz@chromium.org> Reviewed-by:
Nico Hartmann <nicohartmann@chromium.org> Reviewed-by:
-
Clemens Backes authored
This is a reland of 1786f8d7. It turned out that also x64 is broken, and only for TurboFan. Both is fixed now. Original change's description: > [arm64][liftoff] Fix trap handling on load lane > > This fixes the registered {protected_load_pc} to (always) point to the > actual load instruction. If {dst != src} we would emit a register move > before the load, and the trap handler would then not recognize the PC > where the signal occurs, leading to a segfault. > > R=thibaudm@chromium.org > > Bug: chromium:1242300, v8:12018 > Change-Id: I3ed2a8307e353fd85a7ddedf6ecb73e90a112d32 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3136454 > Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Cr-Commit-Position: refs/heads/main@{#76642} Bug: chromium:1242300, v8:12018 Change-Id: I79284ab9815f5363f759569d98c8c4b52d48e738 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140609Reviewed-by:
Maya Lekova <mslekova@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#76698}
-
Rakhim Khismet authored
We add table operations to the fuzzed module. GetTableType function is added in WasmModuleBuilder. Alter alternatives array in GenerateOptRef to generate less default values. Bug: v8:11954 Change-Id: I433a6fac0ab10307aeede505b7d0c1d625d27477 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3137493Reviewed-by:
-
Junliang Yan authored
Change-Id: I1d743994115e659d3f6bf0ce17938b40d1467e5f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144914Reviewed-by:
-
Al Muthanna Athamina authored
Bug: v8:12028 No-Try: true Change-Id: Icc6775ed09fda6b6c46ec08f13137bf5a74662a9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3145410 Auto-Submit: Almothana Athamneh <almuthanna@chromium.org> Commit-Queue: Almothana Athamneh <almuthanna@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#76695}
-
Milad Fa authored
Port 787bec09 Original Commit Message: Remove the BaselineData intermediate structure for baseline code, and write the baseline Code object into the SharedFunctionInfo directly. We still need a pointer to the BytecodeArray/InterpreterData, so re-use the Code object's deoptimization data slot for this (baseline code doesn't have deoptimization data). A consequence of this is that the BytecodeArray pointer becomes immutable when there is baseline code. This means that we cannot install a debug BytecodeArray while baseline code is active (we have to flush it first), and we can't tier-up code with debug BytecodeArray to baseline. R=leszeks@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I7e0f7d964341774340d27f1890cef4c7d9ee9dda Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3145171Reviewed-by:
-
Dan Elphick authored
Removes include now that chrome is not transitively including the header via v8-inspector.h. Bug: v8:11965 Change-Id: Ice039f54f2b97e6fba8765c5220d3145c10ba073 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141585 Auto-Submit: Dan Elphick <delphick@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
-
Michael Lippautz authored
Fix delegation to the specific slow-path bailout. Note: This was not an issue in Blink production code but only when using `cppgc_enable_object_names = true`. Bug: chromium:1056170 Change-Id: I00db63f015b60ac2ccd9f80eca80728bc78e9187 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144911Reviewed-by:
-
Thibaud Michaud authored
This is a reland of 4936628e Original change's description: > [wasm][eh] Ship exception handling > > R=ahaas@chromium.org > CC=ecmziegler@chromium.org > > Cq-Include-Trybots: luci.v8.try:v8_linux_blink_rel > Bug: chromium:1173515 > Change-Id: Id487ec32de7ec10ba1f61f77414565949b8d3dc7 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138203 > Reviewed-by: Andreas Haas <ahaas@chromium.org> > Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> > Cr-Commit-Position: refs/heads/main@{#76672} Bug: chromium:1173515, v8:12173 Change-Id: I61a216207e4aadfd00a716e3296af2f7c3e8df88 Cq-Include-Trybots: luci.v8.try:v8_linux_blink_rel Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141609Reviewed-by:
Georg Neis <neis@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/main@{#76691}
-
Michael Achenbach authored
Bug: chromium:1136844 Change-Id: I635978a308eed2be33c1147f8007abb199bf195e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3143998Reviewed-by:
Almothana Athamneh <almuthanna@chromium.org> Reviewed-by:
Liviu Rau <liviurau@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#76690}
-
Manos Koukoutos authored
We introduce basic wasm inlining infrastructure behind a flag. The implementation is currently incomplete. Additionally, we always inline the function at index 0; proper inlining heuristics will be added later. Changes: - Rename WasmInliningPhase -> JSWasmInliningPhase - Introduce WasmInliningPhase and WasmInliner. - Pass additional parameters as needed to GenerateCodeForWasmFunction. - Remove EnsureEnd in WasmGraphAssembler. Create end node at the start of compilation. - Add a simple test. Bug: v8:12166 Change-Id: Ifd7006ba378e9f74cd248b71e16869fbbb8a82be Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141575 Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by:
-
Liu Yu authored
Port: 787bec09 Change-Id: I40055181a3b14929559672845215ec560906ca8a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144790 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/main@{#76688}
-
Michael Achenbach authored
No-Try: true Bug: v8:12172 Change-Id: I4bace3c91e71e4529ac8d75df3297036c6ca9756 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141588 Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Georg Neis authored
... by only depending on "relevant" results for predicates. Bug: v8:7790, v8:12173 Change-Id: I60b33a3a05197ca7e6d6a36e85c63fd7a48ee931 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3143994Reviewed-by:
-
Georg Neis authored
... as the main thread might change its contents. Bug: v8:12174, v8:7790 Change-Id: I66b2cafc7ddc9b387223693595a9d810b272d7b9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141586Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/45b194e..4f80705 COnsole reporter: if statistic produces percents, format it as such (#1221) (Roman Lebedev) https://chromium.googlesource.com/external/github.com/google/benchmark/+/4f80705 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: I7461c9450c75ad308d26ab478b0d5cc6e5303a64 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144353Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/c6c14d2..03a089b TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I2e17cd400ac5a22a977d78d29e35e223488c41c5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3142000Reviewed-by:
-
Lu Yahan authored
Change-Id: I1937469fdd3aed9cba6ef13b22c5d6f507df4174 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3144114 Commit-Queue: Yahan Lu <yahan@iscas.ac.cn> Commit-Queue: Ji Qiu <qiuji@iscas.ac.cn> Auto-Submit: Yahan Lu <yahan@iscas.ac.cn> Reviewed-by:
-
Lu Yahan authored
Bug: v8:12180 Change-Id: Id3cc3a78da73b10854fd21f2760b25de91ca5966 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3143811Reviewed-by:
-
- 06 Sep, 2021 3 commits
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/db65769..c6c14d2 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/4b2a96e..d579cbc Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/403247e..749fd8e Rolling v8/tools/luci-go: git_revision:7f42370cb3b75398bdb9ae0aabe215a70d40cd31..git_revision:3e1f1f7a109ed8aefc7feba94fa737f0b5b4847e Rolling v8/tools/luci-go: git_revision:7f42370cb3b75398bdb9ae0aabe215a70d40cd31..git_revision:3e1f1f7a109ed8aefc7feba94fa737f0b5b4847e Rolling v8/tools/luci-go: git_revision:7f42370cb3b75398bdb9ae0aabe215a70d40cd31..git_revision:3e1f1f7a109ed8aefc7feba94fa737f0b5b4847e TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I9dfca5f1aedb89819bb65458db57de42a0313eea Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3142337Reviewed-by:
-
Michael Achenbach authored
In order to deprecate different default flags, this sets the flag passed on the standard runner now also on numfuzz. No-Try: true Bug: v8:12177 Change-Id: I3fb6872643f5bfad71362f22a804d22907641c84 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3143992Reviewed-by:
-
Omer Katz authored
We need to check whether ephemeron key is fully constructed to sync the mark bit before checking it. Bug: chromium:1246730 Change-Id: I3ba69898202c1df94833a0bc7442b2be0e61694e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3143993Reviewed-by:
-
