- 06 Oct, 2021 30 commits
-
-
Frank Tang authored
https://chromium.googlesource.com/external/github.com/tc39/test262/+log/50dd431..61339fd2 Bug: v8:7834, v8:12288, v8:11544, v8:11989 Change-Id: Ie7c799f1c757554bd66619fe65433921d4419d52 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3206773Reviewed-by: Shu-yu Guo <syg@chromium.org> Commit-Queue: Frank Tang <ftang@chromium.org> Cr-Commit-Position: refs/heads/main@{#77270}
-
Milad Fa authored
Port 7c08633b Original Commit Message: Replace 'virtual' by 'override' when overriding methods. This uncovered one method which was unnecessarily virtual: {RegExpMacroAssemblerARM64::CheckCharacters}. R=clemensb@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I542aeae836b5b78284291ed39844a5c166ed06ad Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3208811Reviewed-by: Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#77269}
-
Dominik Inführ authored
We are going to introduce safepoints across multiple isolates, with this the name GlobalSafepoint might be misleading. Use IsolateSafepoint as name to emphasise this class reaches a safepoint for a single isolate only. No functional changes. Bug: v8:11708 Change-Id: I8254031dd0bc8e6dcf9f7353297803c37dba47ed Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3207901 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#77268}
-
Clemens Backes authored
Marking the labels as unused is only needed when we abort code generation. Otherwise the DCHECKs in the label destructors are useful to catch bugs. R=jgruber@chromium.org Bug: v8:12244 Change-Id: I63198f98a7acd1f2528d31964c01bc6815ba99a9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3205899 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#77267}
-
Vasili Skurydzin authored
Change-Id: I49a4e2f05028279cd69d3909a9ca80f4c0acb1c5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3208649Reviewed-by: Milad Fa <mfarazma@redhat.com> Reviewed-by: Omer Katz <omerkatz@chromium.org> Commit-Queue: Vasili Skurydzin <vasili.skurydzin@ibm.com> Cr-Commit-Position: refs/heads/main@{#77266}
-
Milad Fa authored
Liftoff is temporarily disabled on PPC. After https://crrev.com/c/3202593 the newly skipped tests are failing with this error: ``` Check failed: tester.native_module()->GetCode(0)->is_liftoff() ``` Change-Id: I681a27930909fd6ac4e5087c2d03608b891a6066 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3208070Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#77265}
-
Manos Koukoutos authored
Bug: v8:7748 Change-Id: Ia277cf58a0eea431c4f19198914c1539fd03bc06 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3207898 Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#77264}
-
Manos Koukoutos authored
Bug: v8:7748 Change-Id: I5b0200560c5d77c1218142ca686c4e71c5106600 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3208027 Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#77263}
-
Michael Achenbach authored
This reverts commit 8860a022. Reason for revert: https://crbug.com/v8/12292 Original change's description: > Update V8 DEPS. > > Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/ebad853..98d97fd > > Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/9959b06..fd29545 > > Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/a002c72..15999e7 > > Rolling v8/third_party/aemu-linux-x64: FAd7QuRV-mCjbKgg2SO4BBlRCvGIsI672THjo3tEIZAC..3M11fN4tUefKOZExDkJbUXnPiJICCo0TwPwoejHowg8C > > Rolling v8/third_party/android_platform: https://chromium.googlesource.com/chromium/src/third_party/android_platform/+log/7a11b79..1a68ade > > Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/c0b9d25..4b10835 > > Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/0e2fb33..281edf7 > > Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/3b49be0..075810f > > Rolling v8/third_party/zlib: https://chromium.googlesource.com/chromium/src/third_party/zlib/+log/dfa96e8..bffc82b > > Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/c06edd1..8640ca8 > > Rolling v8/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:ec7a06c4fcde59629839f8e89c52a08d647e75f4 > > Rolling v8/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:ec7a06c4fcde59629839f8e89c52a08d647e75f4 > > Rolling v8/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:ec7a06c4fcde59629839f8e89c52a08d647e75f4 > > TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com > > Change-Id: I007b32f3475eb71678e50dbf4efa15a88ec85ff6 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3206022 > 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/main@{#77247} Bug: v8:12292 Change-Id: I637676d821c9687fc25ce5b42700a018364f9b7a No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3207902 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/main@{#77262}
-
Camillo Bruni authored
Use a url-like source position format for emited defnition locations. Hopefully this will make links clickable on codesearch. Change-Id: I343c6bc3cc4f159d5e3974d7ec5af4a578aaf03a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3207887Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#77261}
-
George Wort authored
This is a reland of 16df1dfa No changes have been made to this reland as previous commit was reverted due to a new test revealing an existing bug. This bug has now been fixed. Original change's description: > [arm64][wasm-simd] Use Cm(0) for integer comparison with 0 > > Use an immediate zero operand for integer comparison when possible. This > gives ~1% runtime performance improvement in some benchmarks on Neoverse > N1. > > Change-Id: I727a8104f8e6ca3d122d6b5b8b3d38d7bdd76c47 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3158327 > Reviewed-by: Zhi An Ng <zhin@chromium.org> > Commit-Queue: Martyn Capewell <martyn.capewell@arm.com> > Cr-Commit-Position: refs/heads/main@{#76847} Change-Id: I77d6923d79407a83becbd39970c6a3f62d3a304d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3178482Reviewed-by: Zhi An Ng <zhin@chromium.org> Commit-Queue: Rodolph Perfetta <rodolph.perfetta@arm.com> Cr-Commit-Position: refs/heads/main@{#77260}
-
Benedikt Meurer authored
This method had been marked as deprecated over a year ago and since then only `0` on every invocation, and it's not used in Chromium anymore. Fixed: chromium:1246908 Change-Id: Id92a48ac6e42608b1bc023187436f57939466b2c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3208031 Commit-Queue: Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Simon Zünd <szuend@chromium.org> Auto-Submit: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by: Simon Zünd <szuend@chromium.org> Cr-Commit-Position: refs/heads/main@{#77259}
-
Nico Weber authored
_ReadWriteBarrier() is deprecated, and <atomic> has been available for a while now. Ports part of https://chromium-review.googlesource.com/c/chromium/src/+/2365092 to v8. No behavior change. Bug: chromium:1255114 Change-Id: I9954193a69dad9396a5e9e7450066382de2fb172 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3204827 Auto-Submit: Nico Weber <thakis@chromium.org> Commit-Queue: Hannes Payer <hpayer@chromium.org> Reviewed-by: Hannes Payer <hpayer@chromium.org> Cr-Commit-Position: refs/heads/main@{#77258}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/jinja2: https://chromium.googlesource.com/chromium/src/third_party/jinja2/+log/6db8da1..ee69aa0 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I60faeae98fc8341d3ddd493a93a3168a13307410 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3208069Reviewed-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@{#77257}
-
Andreas Haas authored
WebAssembly dynamic tiering should be tested with an origin trial. For the origin trial the feature flag value has to be loaded from blink. This CL stores the value of the --wasm-dynamic-tiering flag in the compilation state, from where it gets passed forward to all uses of the flag. The flag value gets loaded from blink when a new NativeModule is created. R=clemensb@chromium.org Bug: v8:12281 Change-Id: Ia26355a665b7dfcdb47144863c1bec296774abb2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3204963 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Reviewed-by: Adam Klein <adamk@chromium.org> Cr-Commit-Position: refs/heads/main@{#77256}
-
Ng Zhi An authored
Bug: chromium:1255096 Change-Id: I809fb4ef5eaa06975a3e24ad0a36d803aa791c32 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3206837 Auto-Submit: Zhi An Ng <zhin@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/main@{#77255}
-
Jakob Gruber authored
The expected assertion is specific to irregexp codegen. Bug: chromium:1255368 Change-Id: I14d033285014727de2e63582ed798fc82570497d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3207892 Auto-Submit: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#77254}
-
Al Muthanna Athamina authored
Bug: v8:12253 No-Try: true Change-Id: I40d2096fb140389ac6b123506d2170e466534918 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3195712 Commit-Queue: Almothana Athamneh <almuthanna@chromium.org> Auto-Submit: Almothana Athamneh <almuthanna@chromium.org> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Reviewed-by: Liviu Rau <liviurau@chromium.org> Cr-Commit-Position: refs/heads/main@{#77253}
-
Clemens Backes authored
Replace 'virtual' by 'override' when overriding methods. This uncovered one method which was unnecessarily virtual: {RegExpMacroAssemblerARM64::CheckCharacters}. R=jgruber@chromium.org Bug: v8:12244 Change-Id: Ia4480b7b234d3d40cc5821c38ef83f74f8421b6b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3204966Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#77252}
-
Andreas Haas authored
With dynamic tiering, typically not all functions of a WebAssembly module get compiled with TurboFan, and therefore the code caching would never get triggered. With this CL code caching is triggered whenever {FLAG_wasm_caching_threshold} bytes of TurboFan code are generated. This new caching event is only triggered when --wasm-dynamic-tiering is enabled. R=clemensb@chromium.org Bug: v8:12281 Change-Id: I939325aea7e4310aa76c936636799661c05d4079 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3202593Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/main@{#77251}
-
Clemens Backes authored
When aborting code generation, we need to call {AbortedCodeGeneration} on the {MacroAssembler} contained in the {RegExpMacroAssemblerARM}. R=jgruber@chromium.org Bug: chromium:1255368 Change-Id: If37351e8f5715e23affd21ad2de8a8eaad3ea094 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3204965Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#77250}
-
Jakob Kummerow authored
Mirroring Code::entry and CodeDataContainer::code_entry_point. Unused for now; will be used very soon. Change-Id: I93b86f0c601a044bb4e6afea642d5d3f721ef73a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3205893Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Auto-Submit: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#77249}
-
Hao Xu authored
Allocate code range close to binary (<2GB) when pointer compression is disabled. And enable short builtin calls if it succeeds. Bug: v8:12045, v8:11527 Change-Id: I1a9d635b243337980fd75883d9802bc0cee75e43 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3069457 Commit-Queue: Hao A Xu <hao.a.xu@intel.com> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#77248}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/ebad853..98d97fd Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/9959b06..fd29545 Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/a002c72..15999e7 Rolling v8/third_party/aemu-linux-x64: FAd7QuRV-mCjbKgg2SO4BBlRCvGIsI672THjo3tEIZAC..3M11fN4tUefKOZExDkJbUXnPiJICCo0TwPwoejHowg8C Rolling v8/third_party/android_platform: https://chromium.googlesource.com/chromium/src/third_party/android_platform/+log/7a11b79..1a68ade Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/c0b9d25..4b10835 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/0e2fb33..281edf7 Rolling v8/third_party/googletest/src: https://chromium.googlesource.com/external/github.com/google/googletest/+log/3b49be0..075810f Rolling v8/third_party/zlib: https://chromium.googlesource.com/chromium/src/third_party/zlib/+log/dfa96e8..bffc82b Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/c06edd1..8640ca8 Rolling v8/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:ec7a06c4fcde59629839f8e89c52a08d647e75f4 Rolling v8/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:ec7a06c4fcde59629839f8e89c52a08d647e75f4 Rolling v8/tools/luci-go: git_revision:a373a19da0fbbbe81b2b684e3797260294393e40..git_revision:ec7a06c4fcde59629839f8e89c52a08d647e75f4 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I007b32f3475eb71678e50dbf4efa15a88ec85ff6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3206022Reviewed-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@{#77247}
-
Frank Tang authored
This reverts commit a1e6efd8. Reason for revert: Break Arm64 Original change's description: > [intl] Fix consistency in Intl API > > Fix several edge cases consistency issues with ICU discovered by test262 test by > using Intl Enumeration API > 1. Work around ICU short coming of always fallback in currency display > name so when the fallback is "none" in DisplayNames, the force fallback > code will produce the correct undefined from the of(currency_code) method. > 2. Always check numbering system is not algorithm based numbering system > to fix DateTimeFormat/RelativeTimeFormat/NumberFormat > resolvedOptions().numberingSystem when the reqested numberingSystem is one > of the numbering systems that we filter out the resources and not supported. > 3. Generalize the iso8601 bit solution in DateTimeFormat and rename it to > alt_calendar bit to also fix DateTimeFormat resolvedOptions report > calendar as "islamic" while requesting "islamic-rgsa". > 4. Work around reporting inconsistency of currency code and display name > in ICU. > > Bug: v8:12209 > Change-Id: Ibd349ee55426fad7d6f20a5e93fb35ff7438e111 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3153576 > Commit-Queue: Frank Tang <ftang@chromium.org> > Reviewed-by: Shu-yu Guo <syg@chromium.org> > Cr-Commit-Position: refs/heads/main@{#77242} Bug: v8:12209 Change-Id: I6b8ac7dc89eda158e29c9d653825cb20a89341aa No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3207379Reviewed-by: Frank Tang <ftang@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#77246}
-
Seth Brenith authored
Bug: v8:7793 Change-Id: I1d5f85921f06e22679053cc7b2fe0a1371562048 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3206270 Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/main@{#77245}
-
Igor Sheludko authored
... capable of computing the forwarding pointer for objects allocated outside of the main pointer compression cage. Drive-by: hoist computation of pointer compression cage base out of certain loops in GC code. Bug: v8:11880 Change-Id: I23efdffd1a237d9eedd0e2975e8e40811417ef31 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3204968Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/main@{#77244}
-
Camillo Bruni authored
IsActive is misleading as the current implementation forces to use v8::Locker for all Isolate access once any Locker has been used in the same process. Bug: chromium:1240851 Change-Id: Ieb2cfa352313b6f2cbec1bafdbc94a3fc718f3d6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3190093Reviewed-by: Dan Elphick <delphick@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#77243}
-
Frank Tang authored
Fix several edge cases consistency issues with ICU discovered by test262 test by using Intl Enumeration API 1. Work around ICU short coming of always fallback in currency display name so when the fallback is "none" in DisplayNames, the force fallback code will produce the correct undefined from the of(currency_code) method. 2. Always check numbering system is not algorithm based numbering system to fix DateTimeFormat/RelativeTimeFormat/NumberFormat resolvedOptions().numberingSystem when the reqested numberingSystem is one of the numbering systems that we filter out the resources and not supported. 3. Generalize the iso8601 bit solution in DateTimeFormat and rename it to alt_calendar bit to also fix DateTimeFormat resolvedOptions report calendar as "islamic" while requesting "islamic-rgsa". 4. Work around reporting inconsistency of currency code and display name in ICU. Bug: v8:12209 Change-Id: Ibd349ee55426fad7d6f20a5e93fb35ff7438e111 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3153576 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by: Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/main@{#77242}
-
Ng Zhi An authored
A mov can be up to 10 bytes, 6 for displacement, 4 for instr. Other instructions (like pshufb) with a complex addressing mode can take 10 bytes too. So adjust the padding for disassembly of hex accordingly. This requires fixing up all the test cases too. Bug: v8:12207 Change-Id: I372d67a818a5dbfe6f49f67047493d7f67b59bcd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3180375Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/main@{#77241}
-
- 05 Oct, 2021 10 commits
-
-
Samuel Groß authored
This is a reland of 75dd3600 crrev.com/c/3205901 should fix the test failures on Fuchsia. Original change's description: > Reland "Turn on v8_enable_virtual_memory_cage for Chromium builds" > > This is a reland of 4fb3eae7 > > crrev.com/c/3202002 fixed the Chromium build issue. > > Original change's description: > > Turn on v8_enable_virtual_memory_cage for Chromium builds > > > > This CL enables the virtual memory cage at compile time by default for > > Chromium builds on x64 and arm64. However, the cage will only be used at > > runtime if the correpsonding Chromium feature is enabled as well. > > > > Bug: chromium:1218005 > > Change-Id: I5a452d299ac950f8ec0f741f6b9a153e57b2a666 > > Cq-Include-Trybots: luci.v8.try:v8_linux64_heap_sandbox_dbg_ng > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3200081 > > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > > Commit-Queue: Samuel Groß <saelo@chromium.org> > > Cr-Commit-Position: refs/heads/main@{#77212} > > Bug: chromium:1218005 > Change-Id: I32b1a4088ca44827ca4f76b5d19b8138875bfc97 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3204950 > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Commit-Queue: Samuel Groß <saelo@chromium.org> > Cr-Commit-Position: refs/heads/main@{#77229} Bug: chromium:1218005 Change-Id: Id258ded659e4abc31f052ff4c57804d4bd9c5ba0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3205897Reviewed-by: Toon Verwaest <verwaest@chromium.org> Commit-Queue: Samuel Groß <saelo@chromium.org> Cr-Commit-Position: refs/heads/main@{#77240}
-
Peter Kasting authored
Bug: chromium:794619 Change-Id: I335291b8ea7a326abbf66df535d3fa98aff9e4fc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3206277 Commit-Queue: Peter Kasting <pkasting@chromium.org> Commit-Queue: Adam Klein <adamk@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by: Adam Klein <adamk@chromium.org> Cr-Commit-Position: refs/heads/main@{#77239}
-
Clemens Backes authored
The error showed when printing the resulting code object, because the tier was neither TurboFan nor Liftoff, even though the code was registered as a standard wasm function (instead of an import wrapper). R=jkummerow@chromium.org Bug: chromium:1254674 Change-Id: I26482fd88d72403393428979abf08e9f60cd8c4c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3202001 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/main@{#77238}
-
Samuel Groß authored
Bug: chromium:1218005 Change-Id: I00168c25921fd71d925c71c7b7b9ddafd392e95e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3205901Reviewed-by: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Samuel Groß <saelo@chromium.org> Cr-Commit-Position: refs/heads/main@{#77237}
-
Maya Lekova authored
This reverts commit 75dd3600. Reason for revert: Breaks V8 roll, failure on Fuchsia: https://ci.chromium.org/ui/p/chromium/builders/try/fuchsia_arm64/964489/overview Original change's description: > Reland "Turn on v8_enable_virtual_memory_cage for Chromium builds" > > This is a reland of 4fb3eae7 > > crrev.com/c/3202002 fixed the Chromium build issue. > > Original change's description: > > Turn on v8_enable_virtual_memory_cage for Chromium builds > > > > This CL enables the virtual memory cage at compile time by default for > > Chromium builds on x64 and arm64. However, the cage will only be used at > > runtime if the correpsonding Chromium feature is enabled as well. > > > > Bug: chromium:1218005 > > Change-Id: I5a452d299ac950f8ec0f741f6b9a153e57b2a666 > > Cq-Include-Trybots: luci.v8.try:v8_linux64_heap_sandbox_dbg_ng > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3200081 > > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > > Commit-Queue: Samuel Groß <saelo@chromium.org> > > Cr-Commit-Position: refs/heads/main@{#77212} > > Bug: chromium:1218005 > Change-Id: I32b1a4088ca44827ca4f76b5d19b8138875bfc97 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3204950 > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Commit-Queue: Samuel Groß <saelo@chromium.org> > Cr-Commit-Position: refs/heads/main@{#77229} Bug: chromium:1218005 Change-Id: I90f3d5e2878f429125c2a2ebde1105a4116c8d1f No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3205895 Auto-Submit: Maya Lekova <mslekova@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@{#77236}
-
Clemens Backes authored
The {wasm_kind} is completely unused, thus remove it before fixing a wrong {CodeKind} for wasm-to-js functions. R=mslekova@chromium.org Bug: chromium:1254674 Change-Id: Ie3d260a7664d9a390d7edc49c2bf0692c8d798d7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3202000Reviewed-by: Maya Lekova <mslekova@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#77235}
-
Marja Hölttä authored
Bug: v8:11111 Change-Id: I784a9d347fa4a21fd38f04b4d4e3a8a4398292c0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3186438 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by: Shu-yu Guo <syg@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#77234}
-
Milad Fa authored
Change-Id: Id4336aae4e8ef8974657a28cb5e8ea66a968c60c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3202474Reviewed-by: Clemens Backes <clemensb@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#77233}
-
Thibaud Michaud authored
The Merge node for merging exceptions into the catch environment had type kWord32, which is not a reference type. Because of this the GC does not visit it and can collect it too early. Change the type to kTaggedPointer. Also change the type of ExceptionLocation() from IntPtr to TaggedPointer for consistency. This one does not affect correctness because the IfException node is already marked as tagged. R=clemensb@chromium.org Bug: v8:12254 Change-Id: I190d48b85f4b889ab083228b8fcedd439090e1de Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3201994Reviewed-by: Clemens Backes <clemensb@chromium.org> Reviewed-by: Michael Stanton <mvstanton@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/main@{#77232}
-
Clemens Backes authored
R=dinfuehr@chromium.org Bug: v8:12278 Change-Id: I54c2e623e80e13b04b9acbb0915d251ab551eec3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3201996Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/main@{#77231}
-