- 07 Jan, 2021 13 commits
-
-
Georg Neis authored
Bug: chromium:1161357 Change-Id: I7a4237fd682689742e67cd1f35e6ef91815386e0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2611249 Auto-Submit: Georg Neis <neis@chromium.org> Reviewed-by:
Mythri Alle <mythria@chromium.org> Commit-Queue: Mythri Alle <mythria@chromium.org> Cr-Commit-Position: refs/heads/master@{#71943}
-
Liu Yu authored
Port: bbb1b345 Bug: v8:10582 Change-Id: I73cb737655bf68a79f0ae9a25adf9041693a1a8d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2614219Reviewed-by:
Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Auto-Submit: Liu yu <liuyu@loongson.cn> Cr-Commit-Position: refs/heads/master@{#71942}
-
Camillo Bruni authored
- Handle "new" deprecation macros - Use default v8_header path: include/v8.h Change-Id: I9597ca80aa6f6ca297629f1788162add15b31f45 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2607726 Auto-Submit: Camillo Bruni <cbruni@chromium.org> Reviewed-by:
Victor Gomes <victorgomes@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/master@{#71941}
-
Sathya Gunasekaran authored
No need to return anything here as the return value doesn't need to be checked. This is a purely side effecting function. Change-Id: I5484086e289d9f5104a17c93950c25f656d5a44b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2605185Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#71940}
-
Sathya Gunasekaran authored
The FastTemplateCache gets filled with undefiend when created and then with holes when it grows causing a mismatch. This hasn't been a problem so far as the FastTemplateCache is always initialized to the max size but it could cause problems in the future if this is changed. Bug: v8:11289 Change-Id: I8dabce101b25d9f0057183c88f507873d973f2a8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2608472Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Commit-Queue: Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#71939}
-
Manos Koukoutos authored
Loop analysis never visited nodes whose marks happened to not change when visiting their outputs in backwards propagation. This CL ensures that each reachable node is visited at least once. Change-Id: I70cd73737c0abe8151d5e23bc50525599fa3ea6a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2581538Reviewed-by:
Georg Neis <neis@chromium.org> Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/master@{#71938}
-
Leszek Swirski authored
This reverts commit 75da4b99. Reason for revert: Speculative revert for ODROID failures (e.g. https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Arm/17863/overview) Original change's description: > Update V8 DEPS. > > Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/dc0b854..e4c5e3e > > Rolling v8/third_party/aemu-linux-x64: eNKL3iFnDydKoCyqA9rVhylE7ud5a_9wRt0b0HFtLvIC..LanwSa4qZkDJRxK05Ym26IRV6bksO-9CZE-3jrgCbV8C > > Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/3f5c581..ae206b8 > > Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/81098e5..b34cd6d > > Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/4fe0180..1b0cdaa > > Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/1283870..e8d041d > > TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com > > Change-Id: I62cfd8a388b9980b458ff001e5604d704b0332bf > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2614110 > Reviewed-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@{#71934} TBR=machenbach@chromium.org,v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Icbcd46c92ad825d0fbad0694f806b8283290ca7a No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2611254Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#71937}
-
Camillo Bruni authored
Change-Id: Icb5047346fbc0a28755a55b5abe507cfd8913a21 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2599741Reviewed-by:
-
Kong, Fanchen authored
Change-Id: Ie15e8a8469f50a507b77234419e582c21730c586 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2604197Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/dc0b854..e4c5e3e Rolling v8/third_party/aemu-linux-x64: eNKL3iFnDydKoCyqA9rVhylE7ud5a_9wRt0b0HFtLvIC..LanwSa4qZkDJRxK05Ym26IRV6bksO-9CZE-3jrgCbV8C Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/3f5c581..ae206b8 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/81098e5..b34cd6d Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/4fe0180..1b0cdaa Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/1283870..e8d041d TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I62cfd8a388b9980b458ff001e5604d704b0332bf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2614110Reviewed-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@{#71934}
-
Zhi An Ng authored
movaps is 1 byte shorter than movapd and is equivalent. Fixed: v8:11116 Change-Id: Ibf32ad5428ac3d55d055f8725fbf2e96433fb97a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2601878 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#71933}
-
Frank Tang authored
https: //chromium.googlesource.com/external/github.com/tc39/test262/+log/51666c531..b2e9dff28 Bug: v8:7834 Change-Id: I4bd6d2990318b0e722f784b0996719ad95a30efc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2600445Reviewed-by:
Shu-yu Guo <syg@chromium.org> Commit-Queue: Frank Tang <ftang@chromium.org> Cr-Commit-Position: refs/heads/master@{#71932}
-
Milad Fa authored
Port bbb1b345 Original Commit Message: Adds support for emitting data with a reloc info to enable support for HeapObjects for immediate deopt args, required by dynamic check maps. In order to do this, a new DATA_EMBEDDED_OBJECT relocinfo type is added. This represents a raw object inserted into the instruction stream. For x64/ia32 it is treated the same as FULL_EMBEDDED_OBJECT, but on Arm/Arm64 this behaves differently since it points directly to the embedded object pointer rather than to an instruction that loads it. R=rmcilroy@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG=v8:10582 LOG=N Change-Id: I949acb69ca6f6a377102eb0ac5f44919d4f7d25b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2612930Reviewed-by:
Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#71931}
-
- 06 Jan, 2021 12 commits
-
-
Milad Fa authored
xxspltib takes in an immediate value of up to 8 bits. We need to make sure input values are not bigger in size. Change-Id: I3574028d43752870564e4690b22bf2e87813ab76 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2613031Reviewed-by:
-
Milad Fa authored
P9 has vector insert instructions which could be used for splatting 64bit inputs. Change-Id: I16165271fd70848699430e1f77a1603d1023da49 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2611044Reviewed-by:
-
Milad Fa authored
Change-Id: I33cbca84ef65c7e93a0c942aabc881648d58ca3a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2607489Reviewed-by:
-
Milad Fa authored
Change-Id: I8ad99913c570c8e7b542f7c2e1be8235c2ebeb71 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2609412Reviewed-by:
-
Ross McIlroy authored
Adds support for emitting data with a reloc info to enable support for HeapObjects for immediate deopt args, required by dynamic check maps. In order to do this, a new DATA_EMBEDDED_OBJECT relocinfo type is added. This represents a raw object inserted into the instruction stream. For x64/ia32 it is treated the same as FULL_EMBEDDED_OBJECT, but on Arm/Arm64 this behaves differently since it points directly to the embedded object pointer rather than to an instruction that loads it. BUG=v8:10582 Change-Id: I384e88af032b832742447482ae7af5c599097ec1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2595295Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Reviewed-by:
Sathya Gunasekaran <gsathya@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#71926}
-
Milad Fa authored
Change-Id: I70e117d38d81cec0e7687714211b1c503075e3ba Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2609411Reviewed-by:
-
Deepti Gandluri authored
This reverts commit 3c3e15a4. Reason for revert: Revert due to Win32 Bot failures Original change's description: > Update V8 DEPS. > > Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/dc0b854..834e584 > > Rolling v8/third_party/aemu-linux-x64: eNKL3iFnDydKoCyqA9rVhylE7ud5a_9wRt0b0HFtLvIC..9zubYTHwbH6DlxoyH3dV8sX48bSr3IkL8MkHYKlEtVkC > > Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/3f5c581..6c87604 > > Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/81098e5..ac96016 > > Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/1283870..e8d041d > > TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com > > Change-Id: If60b5bc293604bb8dfb0833df43620f743194c15 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2612435 > Reviewed-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@{#71923} TBR=machenbach@chromium.org,v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I143f7b2c8350e98cdb14e6e533ec5bbc3559d907 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2613345Reviewed-by:
Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#71924}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/dc0b854..834e584 Rolling v8/third_party/aemu-linux-x64: eNKL3iFnDydKoCyqA9rVhylE7ud5a_9wRt0b0HFtLvIC..9zubYTHwbH6DlxoyH3dV8sX48bSr3IkL8MkHYKlEtVkC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/3f5c581..6c87604 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/81098e5..ac96016 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/1283870..e8d041d TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: If60b5bc293604bb8dfb0833df43620f743194c15 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2612435Reviewed-by:
-
Zhi An Ng authored
This reverts commit 94f2212b. Reason for revert: Failures https://ci.chromium.org/p/v8/builders/ci/V8%20Win32%20-%20debug/29008? Original change's description: > [wasm-simd] Scalar lowering for extended multiply > > R=bbudge@chromium.org > > Bug: v8:11262 > Change-Id: Idd6a7514a16c561832af603dbf63779a0e402f45 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2603771 > Reviewed-by: Bill Budge <bbudge@chromium.org> > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71920} TBR=bbudge@chromium.org,zhin@chromium.org Change-Id: Ic32ac37ee630c644c1f0605cfa3ca7842c52ce15 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:11262 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2612544Reviewed-by:
Zhi An Ng <zhin@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#71922}
-
Zhi An Ng authored
For i64x2.ext_mul and i16x8.ext_mul, we can simply call the macro-assembler functions. For i32x4.ext_mul, the macro-assembler requires that dst == src1 if AVX is not supported, so we add a helper function to do that check, and make sure dst == src1 before calling into macro-assembler. This is the same implementation as x64 (https://crrev.com/c/2603765). Bug: v8:11262 Change-Id: I85ebe3e033903aa1cb6a7a0840f2e93210cce65a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2610506Reviewed-by:
-
Zhi An Ng authored
R=bbudge@chromium.org Bug: v8:11262 Change-Id: Idd6a7514a16c561832af603dbf63779a0e402f45 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2603771Reviewed-by:
-
Zhi An Ng authored
This adds a regression test for a bug in lowering load transforms. This test will fail if 0efa3fd9 is reverted. Bug: chromium:1124885 Change-Id: I31b714d4565c4fff730c1274af8059031cb1e1b5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2610508Reviewed-by:
-
- 05 Jan, 2021 15 commits
-
-
Sara Tang authored
Design doc: https://docs.google.com/document/d/1xkXj94iExFgLWc_OszTNyNGi523ARaKMWPZTeomhI4U A lot has changed since the last patchset! I recommend revisiting this design doc and reading the parts in green. I explain the roadmap for what changes to expect from ETW instrumentation as well as the instrumentation of this particular CL. I'll do my best to answer any further questions anyone has about my particular instrumentation or ETW in general :) --- This is the first of a series of changelists to round out ETW instrumentation for V8. This changelist represents the most minimal change needed to instrument ETW in V8. In particular, it: - defines and registers the ETW provider, - interacts minimally with the rest of V8, by hooking into the existing TracingController::AddTraceEvent function, - is designed with a platform-agnostic layer, so that event tracers for other platforms can be instrumented in teh future. Some notes on instrumentation (aka I copied stuff from the design doc): We make heavy use of the TraceLogging API to log events. It differs from previous methods of emitting ETW events in that it doesn<E2><80><99>t require the overhead of a separate manifest file to keep track of metadata; rather, events using this API are self-descriptive. Here are the five major steps to instrument the TraceLogging API: - Forward declare the provider (from provider-win.h) - Define the provider in a .cc file (from provider-win.cc) - Register the provider (called from v8.cc). - Write events (called from libplatform/tracing-controller.cc) - Unregister the provider (called from v8.cc) At the base, we have an abstract provider class that encapsulates the functionality of an event provider. These are things like registering and unregistering the provider, and the actual event-logging. The provider class is split into provider-win and provider-mac (currently not instantiated) classes, with OS-dependent implementations of the above functions. In particular, the TraceLogging API is used only in provider-win. It is here that we forward declare and define the provider, as well as write ETW events. Finally, there is a v8-provider class that serves as a top-level API and is exposed to the rest of V8. It acts as a wrapper for the platform-specific providers. The .wprp file is needed so that Windows Performance Recorder knows how to capture our events. Some considerations: - Is TracingController::AddTraceEvent the best place from which to write my events? - Is src/libplatform/tracing the best place to put my instrumentation? - Right now, I fail the preupload because of this, which tells me my files are probably not in the best location: You added one or more #includes that violate checkdeps rules. src\init\v8.cc Illegal include: "src/libplatform/tracing/v8-provider.h" Because of "-src/libplatform" from src's include_rules. Change-Id: Id53e4a034c9e526524a17000da0a647a95d93edf Bug: v8:11043 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2233407Reviewed-by:
Peter Marshall <petermarshall@chromium.org> Commit-Queue: Sara Tang <sartang@microsoft.com> Cr-Commit-Position: refs/heads/master@{#71918}
-
Andreas Haas authored
The negative implication from --single-threaded to --liftoff was introduced because with --single-threaded, no tier-up to TurboFan is possible, and TurboFan provides faster code and smaller code This CL removes this implication. The reason is that this implication does not only define the default compiler for --single-threaded to TurboFan, and also completely disables the option to use Liftoff instead. By removing the implication, and embedder who uses --single-threaded can decide by themselves if they want to use Liftoff by setting --liftoff, or if they don't want to use Liftoff by setting --no-liftoff. R=clemensb@chromium.org Bug: v8:11279 Change-Id: Ie6e64a42823b87484135364ecb4589cfd188db5e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2599548Reviewed-by:
Clemens Backes <clemensb@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#71917}
-
Clemens Backes authored
This reverts commit 2d5f981a. Reason for revert: Fails on noavx: https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20-%20debug/35318 Original change's description: > [wasm-simd][liftoff][x64] Move v128.select into macro-assembler > > This allows us to reuse this optimized code sequence in Liftoff. > > We can't do the same thing in IA32 yet, there is no kScratchDoubleReg > defined in the macro-assembler-ia32.cc, it is defined in code-generator-ia32 > as xmm0 but I'm not sure if it is safe to just use that in the macro assembler. > > Change-Id: I6c761857c49d2518fbc82cd0796c62fc86665cb5 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2596581 > Commit-Queue: Zhi An Ng <zhin@chromium.org> > Reviewed-by: Clemens Backes <clemensb@chromium.org> > Reviewed-by: Bill Budge <bbudge@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71915} TBR=bbudge@chromium.org,clemensb@chromium.org,zhin@chromium.org Change-Id: I2aacee02c89a16516a9cd6686d8cc6180362f78e No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2610730Reviewed-by:
-
Zhi An Ng authored
This allows us to reuse this optimized code sequence in Liftoff. We can't do the same thing in IA32 yet, there is no kScratchDoubleReg defined in the macro-assembler-ia32.cc, it is defined in code-generator-ia32 as xmm0 but I'm not sure if it is safe to just use that in the macro assembler. Change-Id: I6c761857c49d2518fbc82cd0796c62fc86665cb5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2596581 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
-
Clemens Backes authored
The tests were failing in official release builds, because those drop the fatal error message and always print "ignored" instead. R=ecmziegler@chromium.org Bug: v8:11251 Change-Id: I40512ca308337cf070ecb6a206dc4a5323d67415 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2595445Reviewed-by:
Emanuel Ziegler <ecmziegler@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#71914}
-
Clemens Backes authored
According to the Torque definition, the type of the 'source' field is 'String|Undefined'. The Factory only allowed to pass a string though, which forced us to set an empty string for wasm scripts. This CL changes the factory to also allow undefined values, which fits slightly better for wasm. The inspector needed a minor change to handle undefined source like an empty string. R=dinfuehr@chromium.org, yangguo@chromium.org Bug: chromium:1125986 Change-Id: Iac0a5ee3767ce121aba8a6a2afe37195e77122fa Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2584243Reviewed-by:
Dominik Inführ <dinfuehr@chromium.org> Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#71913}
-
Zhi An Ng authored
R=ahaas@chromium.org Change-Id: Iffc27328464616b422cb6ccecbaf77d901f8d889 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2599848 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
Andreas Haas <ahaas@chromium.org> Reviewed-by:
-
Benedikt Meurer authored
Previously the proxies that make up the DebugEvaluate implementation for Wasm frames lived in wasm-js.cc, but that was quite confusing since (a) the rest of the debug support for Wasm lives in wasm-debug.cc (and we intend to eventually unify the DebugEvaluate and Scope objects), and (b) the wasm-js.cc file is explicitly about the WebAssembly JS API that's part of the WebAssembly specification, and the DebugEvaluate proxies aren't part of that. Bug: chromium:1162229, chromium:1071432, chromium:1127914 Change-Id: I63016dcace6d8e2af4a03c8eed4f02d464c1dee1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2609418 Commit-Queue: Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Auto-Submit: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by: -
Benedikt Meurer authored
Drive-by-fix: Handle duplicate globals names correctly in the scope exposed module object. Bug: chromium:1127914, chromium:1071432 Change-Id: I697256642c5ddbc13f86ff25ab012c53537b9c88 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2609416 Commit-Queue: Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Auto-Submit: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
-
Benedikt Meurer authored
The wasm-scope-info-liftoff.js and wasm-set-breakpoint-liftoff.js tests were originally testing the Liftoff path (when we still had the Wasm interpreter), and have received some updates along the way. Nowadays the interpreter is going and the non-liftoff versions of these tests don't provide any additional test coverage, but are merely a slightly less updated version of the liftoff test. Bug: chromium:1162229 Change-Id: Ifc9933d47f33674a83b99425ef9d0e4bc5550323 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2609415 Auto-Submit: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
-
Benedikt Meurer authored
Consistently use InspectorTest.runAsyncTestSuite() in wasm inspector tests to make tests easier to debug (they'll fail instead of timing out in case of errors). Bug: chromium:1162229, chromium:1071432 Change-Id: I7aada196f9e34071aa1bb059bb45f85f75226060 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2609414 Commit-Queue: Yang Guo <yangguo@chromium.org> Auto-Submit: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
-
Zhi An Ng authored
This will make these functions usable from Liftoff when we later implement extended multiply instructions in Liftoff. This is similar to the x64 versions, except that it takes a scratch register as a parameter. Bug: v8:11262 Change-Id: Ief3d8cdde59da9e05a468286315bcae6d13863d9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2603768Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/46adc39..dc0b854 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/364205c..81098e5 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ic95d3b8d0d4265a2f43f6434469e9e65601da52a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2610430Reviewed-by:
-
Zhi An Ng authored
These optimized macro-assembler functions were introduced in https://crrev.com/c/2591853. Reuse them in Liftoff to get better code-gen, especially when AVX is supported. The macro-assembler functions have no alias requirements, so we can pass dst and src directly without doing any checks or moves. Change-Id: I0d1badfc2e2b41a8d92d8c4ac73541ec46daeb0a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2603766Reviewed-by:
-
Zhi An Ng authored
In https://crrev.com/c/2591859 we changed the way we generate code for v128.select, which assumes that all inputs are registers. We did not update the instruction selector with this new constraint. Fixed: chromium:1161954 Bug: v8:11282 Change-Id: I5fc9a0315873a3e795078997d87aa92d4c8bddfe Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2603764 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
-
