- 24 Sep, 2019 18 commits
-
-
Ng Zhi An authored
This gives nicer error messages. Was Fatal error in ../../src/compiler/backend/instruction.h, line 778 Debug check failed: i < InputCount(). Now Fatal error in ../../src/compiler/backend/instruction.h, line 778 Debug check failed: i < InputCount() (5 vs. 3). Bug: v8:9396 Change-Id: Iab2aea49245a42397bf07f4abf9e3de501c758b9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817258Reviewed-by:
Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#63954}
-
Andreas Haas authored
This CL fixes a memory leak in the interpreter. The leak was caused by a cycle the object graph that was rooted with a global object. The cycle was the following: A global handle, owned by the interpreter -> reference stack of the Interpreter -> ref.func element (WasmExportedFunction) -> WasmInstanceObject -> WasmDebugInfo -> InterpreterHandle -> Interpreter With this CL we get rid of the global handle. Instead we store the stack in the WasmDebugInfo. We then have to load the reference stack every time we enter the Interpreter and want access the reference stack. R=mstarzinger@chromium.org Bug: chromium:1000610 Change-Id: If8995725f7ec35862b2f99a07582c861027daaf1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1800582 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by:
Michael Starzinger <mstarzinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#63953}
-
Santiago Aboy Solanes authored
We are going to change again Tagged to not mean 'Full Tagged'. Then, we have to reintroduce this operator, which was removed in https://chromium-review.googlesource.com/c/v8/v8/+/1594439. Bug: v8:7703 Change-Id: I8df65b76eed67613297b04d67c791cb6580bb49e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817613 Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#63952}
-
Milad Farazmand authored
Port 12a9ee3a Original Commit Message: 32 bit Smis are incompatible with pointer compression so we land disable them before enabling pointer compression in order to separate memory and performance regressions caused by 31 bit Smis from pointer compression change. R=ishell@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= LOG=N Change-Id: I1a353f2d6a682ba27e579a7de1bf7ea2240a6bbd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1822117Reviewed-by:
Junliang Yan <jyan@ca.ibm.com> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#63951}
-
Georg Neis authored
R=mslekova@chromium.org Bug: v8:7790 Change-Id: Ie6cc694ac3d11bdd454c2f6d23e24b4fdd59b00e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1822039Reviewed-by:
Maya Lekova <mslekova@chromium.org> Commit-Queue: Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#63950}
-
Santiago Aboy Solanes authored
We are going to change again Tagged to not mean 'Full Tagged'. Then, we have to reintroduce these operators. These allow to write a tagged uncompressed pointer even if pointer compression is enabled. This CL is basically a revert of https://chromium-review.googlesource.com/c/v8/v8/+/1593301. Bug: v8:7703 Change-Id: Ic1c66690b6bc74f697c9f5bf6d3ff45bb764c268 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817612Reviewed-by:
-
Jakob Gruber authored
This extends suppressions to also handle excess zone allocations in the parser. Bug: chromium:1002510 Change-Id: Id418564d244ed618703e62260de31a6b1a2842f8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1821459 Auto-Submit: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#63948}
-
Dan Elphick authored
Always unmark arrowhead parameters as assigned directly after their initialization as the parser doesn't know when it first sees the "assignment" that it may be in an arrowhead. Bug: chromium:1003403, v8:8510 Change-Id: Iad5a4136d5ec06331fc43b81a809fd72cee2dd65 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815131 Commit-Queue: Dan Elphick <delphick@chromium.org> Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#63947}
-
Joshua Litt authored
Adds support for parsing top level await to V8, as well as many tests. This is the final cl in the series to add support for top level await to v8. Spec is here: https://tc39.es/proposal-top-level-await/#sec-execute-async-module Bug: v8:9344 Change-Id: Ie8f17ad8c7c60d1f6996d134ae154416cc1f31e3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1703878Reviewed-by:
Adam Klein <adamk@chromium.org> Commit-Queue: Joshua Litt <joshualitt@chromium.org> Cr-Commit-Position: refs/heads/master@{#63946}
-
Jakob Gruber authored
This introduces a limit for the interpreter's BacktrackStack to match the limit used by generated code (RegExpStack::kMaximumStackSize). Bug: chromium:1006670 Change-Id: I0b7613698e61257aecca89535ad9109c7e454692 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1821458 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Peter Marshall <petermarshall@chromium.org> Auto-Submit: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Peter Marshall <petermarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#63945}
-
Michael Starzinger authored
This reduces the number of label indices accepted by {br_table} from the full function body size to specifically 65520 labels. Note that TurboFan already had a similar limitation on switches, but caused a crash during compilation up until now. This change just makes the limit explicit and avoids the crash during compilation. R=clemensh@chromium.org TEST=mjsunit/regress/wasm/regress-9759 BUG=v8:9759 Change-Id: I3a9a4406b19a7f98fc36707b3b946be846170a15 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1821457 Commit-Queue: Michael Starzinger <mstarzinger@chromium.org> Reviewed-by:Clemens Backes [né Hammacher] <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#63944}
-
Matheus Marchini authored
Also report code relocation events through the public CodeEventHandler API, so that embedders can expose this information to external profilers, allowing those profilers to correctly translate function names even after they were relocated. R=bmeurer@chromium.org, jgruber@chromium.org, yangguo@chromium.org Change-Id: I8795186f5f5c58ede9054e4b83e2d290d92b6e00 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1802657Reviewed-by:
Yang Guo <yangguo@chromium.org> Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
-
Clemens Hammacher authored
In the {Fixed} variant, the {WasmCodeManagerTest} always reserves 1GB of memory. This makes the test run OOM on many 32-bit platforms. Instead of skipping it selectively, this CL just removes the whole test. It caused a lot of trouble in the past, and needs two test-only methods in the WasmCodeManager. Also, the {Fixed} variant will not be needed any more with the wasm far jump table, since modules can always grow then. Drive-by: Clean up the unittests status file a bit. R=mstarzinger@chromium.org Bug: v8:9477 Change-Id: I5b6f8ed9f800863575c69d49d5df82f21fd23030 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815251Reviewed-by: -
Ulan Degenbaev authored
If a BackingStore is marked as !free_on_destruct, then we don't have to guarantee that there is only one such BackingStore pointing to the underlying buffer. So we can skip costly registration in process-global table of backing stores. Bug: v8:9380,chromium:1002693 Change-Id: Iad1ec5c4811d6c52a9a9d78dd700acf69170db60 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815136 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/master@{#63941}
-
Igor Sheludko authored
This is a reland of 12a9ee3a Fixed arm64 disasm test. Original change's description: > [ptr-compr] Switch to 31 bit Smis on 64-bit architectures > > 32 bit Smis are incompatible with pointer compression so we land disable > them before enabling pointer compression in order to separate memory and > performance regressions caused by 31 bit Smis from pointer compression > change. > > Bug: v8:9767 > Change-Id: I3d4a675df4208f808b1ba6e7816be545eae0dc24 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815249 > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Cr-Commit-Position: refs/heads/master@{#63934} Bug: v8:9767 Change-Id: Ife46a4240141dd89d841eac152032ad6ca471810 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1820939Reviewed-by:
-
Ulan Degenbaev authored
This moves the initialization to JSArrayBuffer::SetupEmpty, which is the proper bottleneck for all paths constructing array buffers. Bug: chromium:1006600,v8:9380 Change-Id: I1887cb867627d69ade20654e5bc372b1ba1ac4e3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815132Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/c05af3e..04fbb76 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/fa002e6..8669b37 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/7cb60e8..ba5bc99 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/235f673..e514199 Rolling v8/tools/swarming_client: https://chromium.googlesource.com/infra/luci/client-py/+log/96f1257..885b3fe TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I755fb986c321cffd5ea4e6a14d31d50bec82f9fe Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817576Reviewed-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@{#63938}
-
Yu Yin authored
port f5ab7d38 https://crrev.com/c/1627539. Change-Id: Ibe721144301bff6d843b6d1861b5a20890a057bb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1818004 Commit-Queue: Yu Yin <xwafish@gmail.com> Reviewed-by:
-
- 23 Sep, 2019 19 commits
-
-
Bill Budge authored
This reverts commit 12a9ee3a. Reason for revert: Breaks arm64 sim bot https://ci.chromium.org/p/v8/builders/ci/V8%20Linux%20-%20arm64%20-%20sim%20-%20debug/17467 Original change's description: > [ptr-compr] Switch to 31 bit Smis on 64-bit architectures > > 32 bit Smis are incompatible with pointer compression so we land disable > them before enabling pointer compression in order to separate memory and > performance regressions caused by 31 bit Smis from pointer compression > change. > > Bug: v8:9767 > Change-Id: I3d4a675df4208f808b1ba6e7816be545eae0dc24 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815249 > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Commit-Queue: Igor Sheludko <ishell@chromium.org> > Cr-Commit-Position: refs/heads/master@{#63934} TBR=ishell@chromium.org,verwaest@chromium.org Change-Id: Icc5a5b676a7f7d3fdb3fe3cf84007d4f9bce6eaf No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:9767 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815842Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#63936}
-
Santiago Aboy Solanes authored
Since we were already loading the undefined value a few lines above, we could avoid loading it again. Bug: v8:9771 Change-Id: Id05a77494c06a212612a365b5c0dcba4803ba756 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817609 Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by:
Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#63935}
-
Igor Sheludko authored
32 bit Smis are incompatible with pointer compression so we land disable them before enabling pointer compression in order to separate memory and performance regressions caused by 31 bit Smis from pointer compression change. Bug: v8:9767 Change-Id: I3d4a675df4208f808b1ba6e7816be545eae0dc24 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815249Reviewed-by:
-
Joshua Litt authored
Bug: v8:9463 Change-Id: Ie0e04e102b56ffdfb636e94ef293bb0d46e5f4a9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1808485Reviewed-by:
Michael Stanton <mvstanton@chromium.org> Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Joshua Litt <joshualitt@chromium.org> Cr-Commit-Position: refs/heads/master@{#63933}
-
Dan Elphick authored
Moves the following functions only used in string builtins out of CodeStubAssembler: StringAdd AllocateConsString StringFromSingleUTF16EncodedCodePoint BranchIfCanDerefIndirectString DerefIndirectString MaybeDerefIndirectString MaybeDerefIndirectStrings Bug: v8:9396 Change-Id: Ib89966b9c170ca23dd7535a0f550c69966a6e21c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817608 Commit-Queue: Dan Elphick <delphick@chromium.org> Reviewed-by:
-
Georg Neis authored
Change-Id: I4ddb49bb4cb2cda52247354c53386369ad0e0844 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817610Reviewed-by:
-
Dan Elphick authored
Move CodeStubAssembler::StoreJSTypedArrayElementFromTagged, SetJSTypedArrayOnHeapDataPtr and SetJSTypedArrayOffHeapDataPtr into TypedArrayBuiltinsAssembler. Bug: v8:9396 Change-Id: I16e6aa82d0de9bd1d29a9e2c095032f2c6dd5e29 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817611Reviewed-by:
-
Dominik Inführ authored
Since invalidation is disabled at the moment, slots still need to be cleared explicitly. This was not the case in MigrateFastToFast in case a slot transitioned from tagged to untagged. Bug: chromium:1006630 Change-Id: I8d32ab9196603dabad9a382ec49b39144624c30d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815248Reviewed-by:
-
Ulan Degenbaev authored
Bug: chromium:1006629,v8:9380 Change-Id: I8e45759fe3ad1b0ef8f1ebdb33919c84e1e8a044 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815244Reviewed-by:
-
Dominik Inführ authored
This reverts commit c271cb74. Reason for revert: CL is not needed anymore. Original change's description: > [heap] Emit debugging data before crashing > > We see crashes in the wild, however the minidump does not contain enough > information for properly investigating this. This is temporary code > that should be deleted as soon as possible. > > Bug: chromium:1004365 > Change-Id: I25e119d891e03f980d63f5d559fcb4429b574cb1 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1809373 > Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > Cr-Commit-Position: refs/heads/master@{#63873} TBR=ulan@chromium.org,dinfuehr@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:1004365 Change-Id: Ic8765790bfc96ec144ad02dbdf84e53c6f3e6571 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815252Reviewed-by:
Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/master@{#63927}
-
Michael Starzinger authored
This fixes how arguments of a call to {fround} are being parsed. It now accepts a single "AssignmentExpression" only instead of an "Expression" which could potentially be a whole comma-separated list of expressions. R=clemensh@chromium.org TEST=mjsunit/regress/regress-crbug-1006592 BUG=chromium:1006592 Change-Id: Ifaf0c2b048e4ec18429cc6039c0e7dcdecc1d0bb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815255Reviewed-by: -
Clemens Hammacher authored
This is a very minor cleanup, which makes the output of --trace-wasm-compiler more compact. R=mstarzinger@chromium.org Change-Id: I6b941f32e2ec6c3963e14dd376f8a1d65cf46ec6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815254Reviewed-by:
-
Michael Starzinger authored
This loads the call builtin from the Isolate root instead of embedding it into the instruction stream. This can be more efficient, but more importantly it fixes an issue with tracing and eventually allows for background compilation of these wrappers. R=clemensh@chromium.org TEST=mjsunit/regress/wasm/regress-crbug-1006631 BUG=chromium:1006631 Change-Id: Ife1bc513340d233a3c01789c7b56126fe3b87f6f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815245Reviewed-by:
-
Clemens Hammacher authored
This reverts commit ad83fa92. Reason for revert: Crashes odroid: https://ci.chromium.org/p/v8/builders/ci/V8%20Arm%20-%20debug/11234 Original change's description: > [wasm] Fix TODO and skip test on win32 > > In the {Fixed} variant, the {GrowingVsFixedModule} test first reserves > 1GB of memory, then allocates another 1GB to add it to the module as > code. This uses too much memory on win32, making the test fail. > Before this CL, the {NativeModule} only reserved 128kB upfront (in > contrast to the actual expectation of the test). > > Since all other aspects of this test are already covered by other > tests, we just skip this test on win32. > This allows us to resolve the TODO. > > Drive-by: Clean up the unittests status file a bit. > > R=mstarzinger@chromium.org > > Bug: v8:9477 > Change-Id: I575dd1a3f486e1805e0256e8ee6071246f2c24c4 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1816505 > Reviewed-by: Michael Starzinger <mstarzinger@chromium.org> > Commit-Queue: Clemens Hammacher <clemensh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#63921} TBR=mstarzinger@chromium.org,clemensh@chromium.org Change-Id: Ia9d9b9e311ff8b7524938aeb02543bf2c01bdd27 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:9477 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1815250Reviewed-by:
-
Dan Elphick authored
This tnodifies CodeStubAssembler::Float64ToUint8Clamped and Int32ToUint8Clamped. Bug: v8:6949 Change-Id: I79c8cd45dc4c8333f268cf0a62234d75369f89b6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1813024 Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Auto-Submit: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#63922}
-
Clemens Hammacher authored
In the {Fixed} variant, the {GrowingVsFixedModule} test first reserves 1GB of memory, then allocates another 1GB to add it to the module as code. This uses too much memory on win32, making the test fail. Before this CL, the {NativeModule} only reserved 128kB upfront (in contrast to the actual expectation of the test). Since all other aspects of this test are already covered by other tests, we just skip this test on win32. This allows us to resolve the TODO. Drive-by: Clean up the unittests status file a bit. R=mstarzinger@chromium.org Bug: v8:9477 Change-Id: I575dd1a3f486e1805e0256e8ee6071246f2c24c4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1816505Reviewed-by: -
Dan Elphick authored
Change Parser::AllowsLazyParsingWithoutUnresolvedVariables to return false if it may be parsing an arrow function. Bug: v8:9758, v8:8510 Change-Id: Ic5d213d4358ff954a169c03e449197c3f050880c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1816510Reviewed-by:
-
huangjunming authored
Change-Id: I140b530fcb6332c08d45ef10445619e24987e3ad Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817997Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/5742923..c05af3e Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/82ae4b4..7cb60e8 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/d2e7b20..235f673 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I1432b0b73744d87f669811f253ddb9fa4e34d7b9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817562Reviewed-by:
-
- 22 Sep, 2019 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/2a85930..5742923 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/d4d1ba4..82ae4b4 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I44564080c86c20b9cf3200191b18ed35e3366a6c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817560Reviewed-by:
-
- 21 Sep, 2019 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/9417d7b..2a85930 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/535cd16..fa002e6 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/7735f52..d4d1ba4 Rolling v8/tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang/+log/b6e35ab..d2e7b20 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I276eb908a02e912bdaa9ae79f999314dc10d670d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1817557Reviewed-by:
-
- 20 Sep, 2019 1 commit
-
-
Pavol Marko authored
This reverts commit 0d6aa842. Reason for revert: Suspected to be triggering many test flakes - https://crbug.com/1005723 Original change's description: > [Heap] Allocate descriptor array in young > > This hopefully fix the regression test from c693e005 > > Bug: v8:1004766 > Change-Id: If3c554070af6b795e4b3f99cd592a62453028874 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1809363 > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > Commit-Queue: Victor Gomes <victorgomes@google.com> > Cr-Commit-Position: refs/heads/master@{#63858} TBR=ulan@chromium.org,victorgomes@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:1004766, chromium:1005723 Change-Id: I763036ea6873b4b95783981e9e9f4e79782e7a4b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1816515 Commit-Queue: Adam Klein <adamk@chromium.org> Reviewed-by:
-
