- 16 Aug, 2022 12 commits
-
-
Camillo authored
Change-Id: Ic394381f5159211a437e8ee30a474ed974b7f930 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3832374 Auto-Submit: Camillo Bruni <cbruni@chromium.org> Reviewed-by:
Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/main@{#82477}
-
Camillo authored
Remove the following deprecated functions: include/v8-inspector.h:364 v10.3 Use version with client_is_trusted argument include/v8-locker.h:130 v10.3 This method will be removed. include/v8-message.h:90 v10.3 Use GetHostDefinedOptions include/v8-script.h:51 v10.0 Use HostDefinedOptions include/v8-script.h:671 v10.0 Use CompileFunction Output generated by tools/release/list_deprecated.py. Remove CompileFunctionInContext for chrome and only implement it if V8_SCRIPTORMODULE_LEGACY_LIFETIME is defined. Change-Id: I33dd3665220f484e277e66f340e17ed2c3b49916 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3702449Reviewed-by:
Victor Gomes <victorgomes@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Auto-Submit: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#82476}
-
Frank Tang authored
TotalDurationNanoseconds previously return a double to represent the total duration nanoseconds, but the value could be easily bigger than the precise value a double can represent. A double can precisely represent integer to 2^53, which is only about 104 days if that value is nanoseconds. So we need to change the return type to BigInt. Refactor BalanceDuration to merge common code. Change JSTemporalDuration::Compare to use the BigInt version of TotalDurationNanoseconds Change the call site of TotalDurationNanoseconds in RoundDuration Add newly defined BalancePossiblyInfiniteDuration and change BalanceDuration to call it. Spec text: https://tc39.es/proposal-temporal/#sec-temporal-balancepossiblyinfiniteduration https://tc39.es/proposal-temporal/#sec-temporal-balanceduration https://tc39.es/proposal-temporal/#sec-temporal-totaldurationnanoseconds Split from changes in cl/3750098 Bug: v8:11544 Change-Id: Ia4ca8f9bdba49c3a5e54edeef0d2a5833b0002a2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3824658Reviewed-by:
Adam Klein <adamk@chromium.org> Commit-Queue: Frank Tang <ftang@chromium.org> Cr-Commit-Position: refs/heads/main@{#82475}
-
Omer Katz authored
Make sure there is no background GC when setting flags. Bug: v8:12612, v8:13185 Change-Id: I0a2d4796abe265defa00d86f826003eb048e5bf1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829482 Auto-Submit: Omer Katz <omerkatz@chromium.org> Reviewed-by:
-
Simon Zünd authored
This CL builds upon https://crrev.com/c/3284887 (and partly reverts it). Class literals are a bit iffy when it comes to source position and debugging. Mainly the debugger assumes the following invariant: When we are paused inside a class scope, then we expect the class's BlockContext to be pushed already. On the other hand, when we are paused outside a class scope in a function, we don't expect to find the class's BlockContext. The problem is that there are cases where we can either pause "inside" or "outside" the class scope. E.g.: * `var x = class {};` will break on `class` which is inside the class scope, so we expect the BlockContext to be pushed * `new class x {};` will break on `new` which is outside the class scope, so we expect the BlockContext to not be pushed yet. The issue with the fix in https://crrev.com/c/3284887 is that it adjusted the break position for the bytecode of class literals to ALWAYS be after the BlockContext is pushed. This breaks the second example above. We need to tighten the fix a bit and only defer the break position if the "current source position" is inside the class's scope. This way we always guarantee that the BlockContext is pushed or not, depending if the source position that corresponds to the break position is inside or outside the class's scope. Note 1: The CL updates a lot of the bytecode expectations. This is because the class literals are often the first statement in the snippet so we don't need to defer the break position. Note 2: We add a mirrored debugger test to the inspector test so the fuzzer can have some more fun. Fixed: chromim:1350842 Change-Id: I9b5a409f77be80db674217a685a3fc9f8a0a71cf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827871Reviewed-by:
Shu-yu Guo <syg@chromium.org> Reviewed-by:
Kim-Anh Tran <kimanh@chromium.org> Commit-Queue: Simon Zünd <szuend@chromium.org> Reviewed-by:
Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#82473}
-
Lu Yahan authored
Enable test-gc/RunWasmLiftoff_WasmArrayCop test-gc/RunWasmLiftoff_CastsBenchmark Change-Id: I4d67ce0ead382c2e96951098b0fdd53766e5921f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3822761 Auto-Submit: Yahan Lu <yahan@iscas.ac.cn> Reviewed-by:
Jakob Linke <jgruber@chromium.org> Reviewed-by:
Clemens Backes <clemensb@chromium.org> Commit-Queue: Jakob Linke <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#82472}
-
Feng Yu authored
Bug: v8:12781 Change-Id: I57bda54ef37a9dfdb9f3f8ad7c9e16541b345509 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827130 Commit-Queue: Jakob Linke <jgruber@chromium.org> Reviewed-by:
-
Andreas Haas authored
Once lazy compilation lands we want to continue testing the eager compilation configuration. For that we add --no-wasm-lazy-compilation to the "stress" variant. Bug: v8:12852 Change-Id: I1777aaeb1c8cec0359128ed4b3d34a21aa3ce37e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3810249Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/3424d9d..39f80da R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: If8589b4648f697c47542b434cc3918eb4edb194d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3828992 Bot-Commit: 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@{#82469}
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/6dbdca4..b342107 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/647cfe6..268d645 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: Id4bf0fdb4dfcc136148ccc783bc0520fa69bc006 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3828990 Bot-Commit: 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@{#82468}
-
Lu Yahan authored
Change-Id: I8c02e3b8b73fc46d098993c39bf901f9619c8346 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3830281Reviewed-by:
ji qiu <qiuji@iscas.ac.cn> Commit-Queue: ji qiu <qiuji@iscas.ac.cn> Auto-Submit: Yahan Lu <yahan@iscas.ac.cn> Cr-Commit-Position: refs/heads/main@{#82467}
-
Liu Yu authored
Port commit 1067c6ac Port commit 00746406 Bug: v8:11880, v8:8713, v8:12592 Change-Id: I8787ca38ed8c743f0ee74b2fbd9308b9c8bcb903 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3821204 Commit-Queue: Liu Yu <liuyu@loongson.cn> Auto-Submit: Liu Yu <liuyu@loongson.cn> Reviewed-by:
Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Cr-Commit-Position: refs/heads/main@{#82466}
-
- 15 Aug, 2022 7 commits
-
-
Shu-yu Guo authored
The slow paths of the Array.prototype methods in the change-Array-by-copy proposal don't need to be inlined as macros. Make them builtins where possible. This CL also driveby fixes the fast path for toSpliced to only apply when the returned copy's length is <= kMaxFastArrayLength. Bug: v8:13035 Change-Id: I4034d5d40d7db14d86b33a6f1047e1b17781321a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3830287 Commit-Queue: Shu-yu Guo <syg@chromium.org> Reviewed-by:
-
Feng Yu authored
Bug: v8:12781 Change-Id: I759024fb18ee596ecb678e5b70c95235ea91e520 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827126Reviewed-by:
-
Hao Xu authored
... by selecting better instructions and avoiding sign-extend unsigned smi. Change-Id: I60b47f88dd34bfcda189716ac55d1fab13f3d4a1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3819112 Commit-Queue: Hao A Xu <hao.a.xu@intel.com> Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/main@{#82463}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/24023f3..3424d9d Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/dbab4f8..98014eb R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I66bd70959f326d3edc409e27e3198db0e208e0e1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3831635 Bot-Commit: 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@{#82462}
-
Liu Yu authored
Port commit de04959f Change-Id: Ie7740a09b848f06c87ad096218e3e4e34c2bae3f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3830647 Auto-Submit: Liu Yu <liuyu@loongson.cn> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/buildtools/third_party/libc++/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxx/+log/1f6f8b7..6cc58d6 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: If21e4148c29a89513cc54be48fe77de8733587c0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3831634 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82460}
-
Liu Yu authored
Port commit 8b4272c2 Bug: v8:7700 Change-Id: I9d75fd3341e7bf7d2f0b822b7892683857050594 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3830646 Auto-Submit: Liu Yu <liuyu@loongson.cn> Reviewed-by:
-
- 14 Aug, 2022 2 commits
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/5c91684..24023f3 Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/47ec3a8..dbab4f8 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: Icbdab6cc92afa367f65080e42e3c42bf9835765b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829358 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82458}
-
v8-ci-autoroll-builder authored
Rolling v8/buildtools/third_party/libc++/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxx/+log/9f503be..1f6f8b7 Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/754d016..030b4eb Rolling v8/third_party/fuchsia-sdk/sdk: version:9.20220812.0.1..version:9.20220812.1.1 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I8c483eab1de34fe7498e12d5265613b9c83640bf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829357 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82457}
-
- 13 Aug, 2022 9 commits
-
-
Dominik Inführ authored
Store the is_marking_flag_ in the IsolateData to improve the generated code for the RecordWrite builtin. This allows to load the value of the flag directly using the root register, instead of loading that flag's address first using the root register and only then loading its value. Bug: v8:11708 Change-Id: Id4076a7e519c5f8126e310771c0ccd958dc07278 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3780536 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/36b3854..5c91684 Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/11459ac..47ec3a8 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I0dbf72d26734e3ed1082b530c9e73f8d4d5c66b9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829354 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82455}
-
Omer Katz authored
This is a reland of commit a1b863c1 Original change's description: > [heap] Introduce per-thread storage for concurrent sweeping > > Introduce ConcurrentSweeper as indirection between SweeperJob and > Sweeper to hold per-thread state during sweeping. > This will be used by MinorMC sweeping to hold the pretenuring feedback > map. > > Bug: v8:12612 > Change-Id: Ib363339f9109b405e4cae7f2c08cb4f0eacff8d0 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829466 > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82442} Bug: v8:12612 Change-Id: Ic475d0a8a6e9f9e8b5945bb8d28588db1d524510 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829480Reviewed-by:
Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Auto-Submit: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/main@{#82454}
-
Frank Tang authored
Also change Intl.DateTimeFormat to take temporal objects for format. Design doc: https://docs.google.com/document/d/1gI_Jd9ztnkqhQnDTR4GaI3JOFdMNJW2kQK_UaBPvMug Implement the intl enabled Temporal.Instant.prototype.toLocaleString, Temporal.PlainDate.prototype.toLocaleString Temporal.PlainDateTime.prototype.toLocaleString Temporal.PlainMonthDay.prototype.toLocaleString Temporal.PlainTime.prototype.toLocaleString Temporal.PlainYearMonth.prototype.toLocaleString Temporal.ZonedDateTime.prototype.toLocaleString Implement AOs: HandleDateTimeTemporalDate, HandleDateTimeTemporalYearMonth, HandleDateTimeTemporalMonthDay, HandleDateTimeTemporalTime HandleDateTimeTemporalDateTime, HandleDateTimeTemporalInstant, HandleDateTimeTemporalZonedDateTime, HandleDateTimeOthers, HandleDateTimeValue Spec Text: https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevaluetemporaldate https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevaluetemporalyearmonth https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevaluetemporalmonthday https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevaluetemporaltime https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevaluetemporaldatetime https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevaluetemporalinstant https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevaluetemporalzoneddatetime https://tc39.es/proposal-temporal/#sec-temporal-handledatetimeothers https://tc39.es/proposal-temporal/#sec-temporal-handledatetimevalue https://tc39.es/proposal-temporal/#sup-temporal.instant.prototype.tolocalestring https://tc39.es/proposal-temporal/#sup-temporal.plaindate.prototype.tolocalestring Temporal.PlainDateTime.prototype.toLocaleString https://tc39.es/proposal-temporal/#sup-temporal.plainmonthday.prototype.tolocalestring https://tc39.es/proposal-temporal/#sup-temporal.plaintime.prototype.tolocalestring https://tc39.es/proposal-temporal/#sup-temporal.plainyearmonth.prototype.tolocalestring https://tc39.es/proposal-temporal/#sup-temporal.zoneddatetime.prototype.tolocalestring Remove the output_range parameter and use the is_null() of MaybeHandle<T> for to check the optional return. Bug: v8:11544 Change-Id: Ia9ffe7a71c8fb68391581eb5ef7620708e2bd1f0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3791526 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/icu: https://chromium.googlesource.com/chromium/deps/icu/+log/50ec7b3..b3070c5 [build] Suppress -Wenum-constexpr-conversion (Gulfem Savrun Yeniceri) https://chromium.googlesource.com/chromium/deps/icu/+/b3070c5 Use LTO bug workaround in Fuchsia build (Roland McGrath) https://chromium.googlesource.com/chromium/deps/icu/+/4404a4d R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,ftang@chromium.org Change-Id: I9f7ffb1b34c1068a1577e6bda164da55f28601d2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829353 Bot-Commit: 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@{#82452}
-
v8-ci-autoroll-builder authored
Rolling v8/buildtools/third_party/libc++/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxx/+log/55e0c49..9f503be Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/a097a1a..754d016 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/b2cd647..6dbdca4 Rolling v8/third_party/fuchsia-sdk/sdk: version:9.20220811.2.1..version:9.20220812.0.1 R=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: Ie7fd100c2a03e0351888e4362941264ace1b554b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829352 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82451}
-
Frank Tang authored
Bug: v8:11544 Change-Id: I41662b3c43a1a387f3c11796774999e515b00a32 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3824196 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
Adam Klein authored
Bug: v8:13185 Change-Id: I0c6e4ba8b325c3ac70dbceb927e2a8b1f9d68a16 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3830286 Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Adam Klein <adamk@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82449}
-
Omer Katz authored
This is a reland of commit 924be695 Original change's description: > [heap] Use PagedNewSpace when MinorMC is enabled > > This CL also introduces/updates DCHECKs that some methods are never > reached with MinorMC (they may still be reached by full GC when MinorMC > is disabled). > > Bug: v8:12612 > Change-Id: I8afb8c964bc5c44225a92d0f8d9ac5a4c0ecef75 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3823130 > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82439} Bug: v8:12612 Change-Id: I64aa83d48fb48970ee45263356aaf1541e3d6bdc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827040 Commit-Queue: Adam Klein <adamk@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82448}
-
- 12 Aug, 2022 10 commits
-
-
Frank Tang authored
Also add AOs: DifferenceTemporalPlainDateTime, DifferenceTemporalZonedDateTime, Spec Text: https://tc39.es/proposal-temporal/#sec-temporal.plaindatetime.prototype.since https://tc39.es/proposal-temporal/#sec-temporal.plaindatetime.prototype.until https://tc39.es/proposal-temporal/#sec-temporal.zoneddatetime.prototype.since https://tc39.es/proposal-temporal/#sec-temporal.zoneddatetime.prototype.until https://tc39.es/proposal-temporal/#sec-temporal-differencetemporalplaindatetime https://tc39.es/proposal-temporal/#sec-temporal-differencetemporalzoneddatetime split from cl/3750098 Bug: v8:11544 Change-Id: I326d91a897fcd7e39de2fb2bfecfc5bfd920eb44 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827313Reviewed-by:
-
Adam Klein authored
This reverts commit 924be695. Reason for revert: speculative revert for TSAN failures: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN%20-%20stress-incremental-marking/8726/overview Original change's description: > [heap] Use PagedNewSpace when MinorMC is enabled > > This CL also introduces/updates DCHECKs that some methods are never > reached with MinorMC (they may still be reached by full GC when MinorMC > is disabled). > > Bug: v8:12612 > Change-Id: I8afb8c964bc5c44225a92d0f8d9ac5a4c0ecef75 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3823130 > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82439} Bug: v8:12612 Change-Id: I540f38fea17fbacffbd120dd050626d7d1ec32f3 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827039 Auto-Submit: Adam Klein <adamk@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@{#82446}
-
Frank Tang authored
This is a reland of commit cf765fc3 Original change's description: > [Temporal] Use double instead of int32_t for input of BalanceTime > > To avoid overflow int32_t in the math of balancing time. > > Bug: v8:13182, v8:11544 > Change-Id: Ib76cf95bbd4f9b47efd6921a67b09d3024e72b13 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827310 > Reviewed-by: Adam Klein <adamk@chromium.org> > Commit-Queue: Frank Tang <ftang@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82409} Bug: v8:13182, v8:11544 Change-Id: I7550b3a7186beed0e32e95a41cae87030d0c5a7c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827671 Commit-Queue: Frank Tang <ftang@chromium.org> Reviewed-by:
-
Samuel Groß authored
The capacity may be modified on one thread when growing the table while being used in a DCHECK (to sanity-check a provided ExternalPointerHandle) on another thread, resulting in TSan failures. This CL turns these accesses into atomic accesses and adds a comment explaining when the capacity value can be used reliably. Bug: chromium:1352148 Change-Id: I0b86a47e16cfa14ff2d296e7e507e38a3fb5893c Cq-Include-Trybots: luci.v8.try:v8_linux64_heap_sandbox_dbg_ng,v8_linux_arm64_sim_heap_sandbox_dbg_ng Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3826244 Commit-Queue: Samuel Groß <saelo@chromium.org> Reviewed-by:
-
Nico Hartmann authored
This reverts commit a1b863c1. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Clusterfuzz%20Linux%20ASAN%20no%20inline%20-%20release%20builder/8288/overview Original change's description: > [heap] Introduce per-thread storage for concurrent sweeping > > Introduce ConcurrentSweeper as indirection between SweeperJob and > Sweeper to hold per-thread state during sweeping. > This will be used by MinorMC sweeping to hold the pretenuring feedback > map. > > Bug: v8:12612 > Change-Id: Ib363339f9109b405e4cae7f2c08cb4f0eacff8d0 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829466 > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Reviewed-by: Dominik Inführ <dinfuehr@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82442} Bug: v8:12612 Change-Id: I66865a807908a6ef296e06530f293dcf27fea1a3 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829478 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Owners-Override: Nico Hartmann <nicohartmann@chromium.org> Auto-Submit: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/main@{#82443}
-
Omer Katz authored
Introduce ConcurrentSweeper as indirection between SweeperJob and Sweeper to hold per-thread state during sweeping. This will be used by MinorMC sweeping to hold the pretenuring feedback map. Bug: v8:12612 Change-Id: Ib363339f9109b405e4cae7f2c08cb4f0eacff8d0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829466 Commit-Queue: Omer Katz <omerkatz@chromium.org> Reviewed-by:
-
Patrick Thier authored
to get rid of the pattern ``` EnsureHash(); uint32_t field = raw_hash_field(); ``` which requires an additional load and might be misleading in the presence of forwarding indices for shared strings, as raw_hash_field() can return a forwarding index, whereas EnsureRawHash() will always return a computed hash value. Bug: v8:12957 Change-Id: I33426fef433f774fb323d4381e784c1037fb6fbb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829469Reviewed-by:
-
ishell@chromium.org authored
... to please mksnapshot which expects the new space to be empty. Bug: v8:10470 Change-Id: I7d5b62db138ef2e334581a8697d137cd13291d7c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3827877 Auto-Submit: Igor Sheludko <ishell@chromium.org> Reviewed-by:
-
Omer Katz authored
This CL also introduces/updates DCHECKs that some methods are never reached with MinorMC (they may still be reached by full GC when MinorMC is disabled). Bug: v8:12612 Change-Id: I8afb8c964bc5c44225a92d0f8d9ac5a4c0ecef75 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3823130Reviewed-by:
-
Nico Hartmann authored
This reverts commit a19316d9. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20UBSan/22670/overview Original change's description: > [heap] Rework Worklist base type > > Worklist uses a singly-linked list of segments to hold entries. > Segment size was based on a compile-time constant but already stored > in the segment itself. > > Rework the segments to query `malloc_usable_size()` on allocation and > adjust the capacity properly. For PartitionAlloc, it turns out that > there's ~20% more capacity available for the 64-element segments. > > This slows down actual allocation of the segments with the upside of > improving utilization and requiring 20% less segments. > > Change-Id: Ib8595c3fb9fb75b02e4022f6c525bb59a2df7ab7 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3826047 > Commit-Queue: Anton Bikineev <bikineev@chromium.org> > Commit-Queue: Michael Lippautz <mlippautz@chromium.org> > Reviewed-by: Anton Bikineev <bikineev@chromium.org> > Cr-Commit-Position: refs/heads/main@{#82432} Change-Id: I14994e11ff5ffaba70b93d977d40dd2f6e9e5d35 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3829474 Owners-Override: Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Nico Hartmann <nicohartmann@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/heads/main@{#82438}
-
