- 03 Sep, 2021 10 commits
-
-
Michael Lippautz authored
Before this patch, both key and value of an ephemeron pair was always considered to be GarbageCollected objects. This patch adjusts the snapshotting mechanism to accomodate that values may not be GarbageCollected objects and must thus be eagerly traced for visibility and edge creation. In practice this only shows up in Blink when associating an existing wrappable with a wrapper in a non-main world, e.g., through an extension. In this case, DOMWrapperMap keeps the wrapper value through a TracedReference in the ephemeron map with the existing wrappable as key. The semantics are intended to be general ephemeron semantics, i.e., value needs to be kept alive when the key is alive. This is visualized in DevTools as the main wrapper/wrappable pair (which is merged into a single node for the snapshot) retaining the non-main world wrapper. Bug: chromium:1245894 Change-Id: Ibfa6722f20c76f94c310f9a040f0d3d4b9083bbb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140601Reviewed-by:
Omer Katz <omerkatz@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/main@{#76658}
-
Andreas Haas authored
R=thibaudm@chromium.org Change-Id: Id123af24456ee6a6388085e9168766f45321fa49 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3141570Reviewed-by:
Thibaud Michaud <thibaudm@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/main@{#76657}
-
Omer Katz authored
PretenureAllocationSite didn't check whether the given object is in new space or not. Once given an object in old space, PretenureAllocationSite tried to find a memento for it which didn't exist and crashed. This CL adds a bailout for objects not in new space as there is no memento and nothing to be done. Bug: chromium:1244333 Change-Id: Ic26a6f5994ef9942decda69bb8a23fb730bf945c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140604Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/main@{#76656}
-
Michael Achenbach authored
After concurrent inlining is true by default we keep testing the negated version on the main linux bots and drop testing the variant on FYI, which is a no-op now. Bug: v8:7790 Change-Id: I604838a45f3de242db82b42b93afdb56804152b5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140599 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/main@{#76655}
-
Marja Hölttä authored
ParserBase::ParseClassLiteral and BaseConsumedPreparseData::RestoreDataForScope both declare the class variable, but the logic is so complex that they sometimes ended up both declaring it. This is further complicated by some of the variable values (esp. inner_scope_calls_eval_) potentially changing in between, so we can't just redo the same logic any more. Forcefully make it work by making RestoreDataForScope declare the variable iff ParseClassLiteral didn't. Bug: chromium:1245870 Change-Id: I777fd9d78145240448fc25709d2b118977d91056 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140596 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#76654}
-
Jakob Gruber authored
.. since it is now enabled by default. Bug: v8:7790,v8:12142 Change-Id: Ia13e5ef9c1224b02dfe635c5fcd91e7a0346f5ff Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138196Reviewed-by:
-
Yoshisato Yanagisawa authored
Manually update reclient version to re_client_version:0.40.0.40ff5a5. Bug: chromium:1233780 Change-Id: I0ccae131efa038b8e82427af28ca78e527659237 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140617 Commit-Queue: Yoshisato Yanagisawa <yyanagisawa@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Yoshisato Yanagisawa <yyanagisawa@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#76652}
-
Al Muthanna Athamina authored
Bug: v8:11004 No-Try: true Change-Id: I9fa00095c25bf182faf660046129cb3577ce8199 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138208Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/third_party/google_benchmark/src: https://chromium.googlesource.com/external/github.com/google/benchmark/+log/2b09332..e7a8415 CMake: add forgotten include(FeatureSummary) into FindPFM.cmake to fix build (Roman Lebedev) https://chromium.googlesource.com/external/github.com/google/benchmark/+/e7a8415 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com,mlippautz@chromium.org Change-Id: I67ffb580cb9b3d8f56262b001d8c5468fab1d235 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138604Reviewed-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@{#76650}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/1dfc04f..5c32531 Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/9e8b0c9..7ea3a87 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/9f0517e..7de86cb Rolling v8/buildtools/third_party/libunwind/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind/+log/6474ba4..edf77b2 Rolling v8/third_party/aemu-linux-x64: LrM1UivUqag71JX4WdAnr5pc_zp92frKvtN6GhDs2zEC..zV70YxspSldB66kzaPKeo6zR_1yozZLp4bpWWR8dWRQC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/d9a9ebb..2331f08 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/ae44ef1..8d07f5a Rolling v8/third_party/instrumented_libraries: https://chromium.googlesource.com/chromium/src/third_party/instrumented_libraries/+log/ea83816..47226fa Rolling v8/tools/luci-go: git_revision:e08764bfcf2e87425a025e3a1d196c5740385da2..git_revision:7f42370cb3b75398bdb9ae0aabe215a70d40cd31 Rolling v8/tools/luci-go: git_revision:e08764bfcf2e87425a025e3a1d196c5740385da2..git_revision:7f42370cb3b75398bdb9ae0aabe215a70d40cd31 Rolling v8/tools/luci-go: git_revision:e08764bfcf2e87425a025e3a1d196c5740385da2..git_revision:7f42370cb3b75398bdb9ae0aabe215a70d40cd31 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: Ic35c01b8f299bcb8b0a53b99d08aba7fe161d2a8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138531Reviewed-by:
-
- 02 Sep, 2021 18 commits
-
-
Milad Fa authored
This CL takes advantage of the z15 `load byte reverse element` instruction to optimize Simd LoadLane opcodes. On the simulator we only run `load element` as reversing is not required. Change-Id: I038535f7e038bed7972844806644f50519d4919c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138212Reviewed-by:
Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/main@{#76648}
-
Michael Achenbach authored
No-Try: true Bug: chromium:1222092 Change-Id: I9f662691f1cd1a221162052789a66ea99b98d371 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135579 Auto-Submit: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Almothana Athamneh <almuthanna@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#76647}
-
Luis Fernando Pardo Sixtos authored
Fixed issue were using the `arguments` object as a shorthand for a class field initializer was not producing an early error. Bug: chromium:1216261 Change-Id: I7d8f5a85c6881f7ca12a0e8450954de15bdd6033 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3095017Reviewed-by:
-
Milad Fa authored
Port e6961df2 Original Commit Message: Instrument floating-point operations to set a flag if the result is NaN. R=martyn.capewell@arm.com, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I6f3fe550bf30f85bbb3fa9437d676896e876fd30 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3137418Reviewed-by:
-
Nico Hartmann authored
This reverts commit 1786f8d7. Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64/44442/overview 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: I7bc9d00a4fba3101e7ee68695961d1b543268c4e No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138202Reviewed-by:
Nico Hartmann <nicohartmann@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Cr-Commit-Position: refs/heads/main@{#76644}
-
Yoshisato Yanagisawa authored
Bug: chromium:1233780 Change-Id: I9f8a83233606d55af43e2cc31263440af342e36f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3140034 Auto-Submit: Yoshisato Yanagisawa <yyanagisawa@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Clemens Backes authored
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/+/3136454Reviewed-by: -
Georg Neis authored
Bug: chromium:1245949 Change-Id: Ib0a9b3b9307c925d6c3bb0b9c294eea5545b3819 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138200 Commit-Queue: Georg Neis <neis@chromium.org> Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> Auto-Submit: Georg Neis <neis@chromium.org> Reviewed-by:
-
Jakob Gruber authored
Drive-by: simplifications and avoid a repeated concurrent read. Bug: v8:7790,v8:12157 Change-Id: I460c44853a78bcd9a1427e62a92994ff8602dbed Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3133148 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Georg Neis <neis@chromium.org> Auto-Submit: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
-
Marja Hölttä authored
Bug: chromium:1203122 Change-Id: Ief88320b620dbf2f347bf6f6fc1ebd459e60af3d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138194Reviewed-by:
Igor Sheludko <ishell@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/main@{#76639}
-
Lu Yahan authored
Bug: v8:12151 Change-Id: I51a6ebc1fb94b62acdc018cfcf6fed498f1c9d21 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3139576 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:
Ji Qiu <qiuji@iscas.ac.cn> Cr-Commit-Position: refs/heads/main@{#76638}
-
Patrick Thier authored
- Replace CodeStubArguments::GetLength() with GetLengthWithReceiver() and GetLengthWithoutReceiver() - Introduce and use Torque macros to load the formal parameter count (with and without receiver). - Add actual_count to Torque arguments structure for cases where the argument count is not used, but just forwarded to other builtins. Bug: v8:11112 Change-Id: I32278efeffa2fb08361989c6df8de56c74add8b9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3124804Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Clemens Backes <clemensb@chromium.org> Reviewed-by:
-
Liu Yu authored
Besides, fix an error in set_if_nan, because if src is a NaN, we should set the i32 instead of i64 at address dst to a non-zero value. Port e6961df2 Bug: v8:11856 Change-Id: Icc9afda35d4cca4fd5ae82356ecaec77bf92d009 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3139055Reviewed-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/main@{#76636}
-
Zhao Jiazhong authored
Change-Id: Icf548bcbb619dd6d5fa8ac6f11f9ba01820e4215 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3139481Reviewed-by:
Liu yu <liuyu@loongson.cn> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Cr-Commit-Position: refs/heads/main@{#76635}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/85ec285..1dfc04f Rolling v8/buildtools: https://chromium.googlesource.com/chromium/src/buildtools/+log/ac4ea2d..9e8b0c9 Rolling v8/buildtools/third_party/libc++abi/trunk: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi/+log/be47d0e..9f0517e Rolling v8/third_party/aemu-linux-x64: 8RPB9bFL9jMqtkuzgf6G69QDdAGV9Re0Exh-3rzLXvoC..LrM1UivUqag71JX4WdAnr5pc_zp92frKvtN6GhDs2zEC Rolling v8/third_party/android_platform: https://chromium.googlesource.com/chromium/src/third_party/android_platform/+log/e98c753..6e5dc9a Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/83a41f1..d9a9ebb Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/9c1a533..ae44ef1 Rolling v8/third_party/instrumented_libraries: https://chromium.googlesource.com/chromium/src/third_party/instrumented_libraries/+log/ee10dbd..ea83816 TBR=v8-waterfall-sheriff@grotations.appspotmail.com,mtv-sf-v8-sheriff@grotations.appspotmail.com Change-Id: I90d1c0251372cd4d5ff2ca6db7f28842747b7769 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3138519Reviewed-by:
-
QiuJi authored
Also fix several out of date comments. Change-Id: I15ee6c718ad50f231cd0a8e5c6416ccb58375140 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3121693 Commit-Queue: Ji Qiu <qiuji@iscas.ac.cn> Reviewed-by:
Brice Dobry <brice.dobry@futurewei.com> Cr-Commit-Position: refs/heads/main@{#76633}
-
Liu Yu authored
Acquire a scratch register instead of passing scratch register in JumpIfSmi and JumpIfNotSmi. Change-Id: I21776e1cd9d19f0d55c25a5bb3f7efcc247c80e5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135155Reviewed-by:
-
legendecas authored
Removes outdated type-error throwing on TypedArray.prototype.set when the first argument is a number. Bug: v8:11294 Change-Id: Ida3a46dec154b645620e2b064ded7a18de238649 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3136773Reviewed-by:
Shu-yu Guo <syg@chromium.org> Commit-Queue: Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/main@{#76631}
-
- 01 Sep, 2021 12 commits
-
-
QiuJi authored
Port: 732f394c Change-Id: I1a7a8dd7870a71df9f0e7c30392cd9a23b5cab3e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3126820 Commit-Queue: Brice Dobry <brice.dobry@futurewei.com> Reviewed-by:
-
Pierre Langlois authored
The --turbo-stats and --turbo-stats-wasm flags are useful but they do not work on Chromium on Android, given we cannot print on exit of the renderer process. To cover all scenarios, we can encode the statistics as a string argument in the trace format. It's also helpful to see those statistics, as well as the code kind and function name, when clicking on a slice in chrome://tracing or perfetto. As a drive-by cleanup, rename ESCAPE to QUOTE in the JSON serialization code. Change-Id: I86f03d0e020c9543feb869620164bf1aad3a2432 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3132966Reviewed-by:
-
Georg Neis authored
After manually triggering finalization we should clear a potential pending InstallCode interrupt request as there's nothing else to do and keeping the request would defeat the purpose of the intrinsic. Fixed: v8:12152 Change-Id: I063959c97c31868864a82c0584cabf779750d10d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135578Reviewed-by:
-
Manos Koukoutos authored
Bug: v8:7748 Change-Id: I78a41e593b668f417f1cc18b24bc61a4b6e098c1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135577Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/main@{#76627}
-
Clemens Backes authored
This is a reland of 6ae18c2d, with {CompileWasmCapiCallWrapper} fixed to also contain a {CodeSpaceWriteScope}. Original change's description: > [wasm] Move write scope out of NativeModule::AddCode > > {NativeModule::AddCode} is a central method that should usually be > called in batches, where the caller holds a {CodeSpaceWriteScope} for a > longer time (over several compilations). > This CL moves us closer to that by removing the scope from that central > method and instead putting it in callers where it becomes more visible. > There are already TODOs to introduce caching or batching to avoid some > switching, and one more TODO is added. > > Drive-by: Remove an unneeded {CodeSpaceMemoryModificationScope}. > > R=jkummerow@chromium.org > > Bug: v8:11974 > Change-Id: Ia13c601abc766e5fca6ca053bf1fc4d647b53ed0 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3098186 > Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#76344} Bug: v8:11974 Cq-Include-Trybots: luci.v8.try:v8_mac_arm64_dbg_ng Cq-Include-Trybots: luci.v8.try:v8_mac_arm64_rel_ng Change-Id: I6367bbd9dc52c403513eb1a168aa1f6eb4044ca1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3129703Reviewed-by:
-
Manos Koukoutos authored
This is needed so tables are available for table operations. Bug: v8:11954 Change-Id: If0cbb07ddf0852d2e2515aca3e1f54168c2e0ab8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135576Reviewed-by:
-
Victor Gomes authored
This reverts commit 20dc4dcc. Reason for revert: We're killing SP on the heap. Original change's description: > [baseline] Enable SP on the heap on future > > Bug: v8:11872 > Change-Id: Iaaf3ed68950a6a9fe60f91d73be8a0fbc17e1116 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3129424 > Commit-Queue: Victor Gomes <victorgomes@chromium.org> > Commit-Queue: Leszek Swirski <leszeks@chromium.org> > Auto-Submit: Victor Gomes <victorgomes@chromium.org> > Reviewed-by: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/main@{#76578} Bug: v8:11872, v8:12158 Change-Id: Ib12912250013cdebeca1597f6c9dcd4f763b7a4a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135660 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Cr-Commit-Position: refs/heads/main@{#76624}
-
Jakob Gruber authored
Broken by crrev.com/c/3129420 error: ‘CompilationDependency::AsTransition() const’ defined but not used. Bug: v8:7790 Change-Id: I06839c4d33d3a52909e0e5a276c567eca83e910f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3133147 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
-
Michael Lippautz authored
Other threads may write the stack of a different thread and use a lock to synchronize such an access. An example for this is interrupt handling. Ignore TSAN for the methods performing the stack walk. There's no need to use relaxed atomic reads as same-thread writes are consistent and for other-thread writes there's no guarantee on what values to observe. Bug: chromium:1245409 Change-Id: Ia3d3621590f1f5524d245632a2e8a2db23313f35 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135573 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Anton Bikineev <bikineev@chromium.org> Reviewed-by:
-
Al Muthanna Athamina authored
Bug: v8:12013 Change-Id: I382f505914633598865586bc6a1d5dac7c875994 No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135581 Auto-Submit: Almothana Athamneh <almuthanna@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
-
Pierre Langlois authored
Building with v8_use_perfetto requires that the categories passed to TRACE_EVENT* be a constexpr. Change-Id: Iee4b713d8fe0b3f52f6e5cfe5baef0ced87f9855 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3135575Reviewed-by:
-
Jakob Kummerow authored
When an attempt to parse a huge string to a BigInt fails, then including the entire string in it makes the exception's message unwieldy, so this patch puts only the first 1000 characters of such invalid strings into the exception message. Bug: chromium:1245239 Change-Id: I2c62f0d34256653ba67da9666e8c5a1a4bbe0599 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3133142 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Mathias Bynens <mathias@chromium.org> Cr-Commit-Position: refs/heads/main@{#76619}
-
