- 07 Jan, 2020 30 commits
-
-
Ng Zhi An authored
Bug: v8:9813 Change-Id: Iffa5613f0d4226a25519feab8c2246be8e462cc9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1981073 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#65619}
-
Ng Zhi An authored
Bug: v8:10039 Change-Id: I3568bd3d01508e8bca81959341c75369c5bdf700 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1958051Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#65618}
-
Ng Zhi An authored
Bug: v8:9561 Change-Id: Ie3231038312495c2d8f77062ee5b81b2b55ab4d7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1980502Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#65617}
-
Ng Zhi An authored
Bug: v8:9415 Bug: v8:10021 Change-Id: I77c24b58f575b612e5422bfcb9bb7ab83986659a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1986249Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#65616}
-
Ng Zhi An authored
Bug: v8:10039 Change-Id: If7c9668821a1cdfd5968f1533c3412247567bf3e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1955550Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Zhi An Ng <zhin@chromium.org> Cr-Commit-Position: refs/heads/master@{#65615}
-
Z Nguyen-Huu authored
This is reland of 6ce3046e Skip the test in multi-isolates config. Bug: v8:9654 Change-Id: I035c867c99219bbf8d4fa25b58c734306e25fa63 Original change's description: > [wasm] Add support to tier down/up Wasm NativeModule > > This is the first part of switching between Liftoff and Turbofan in > debugging Wasm. In this CL, we implemented the logic to tier down/up all > functions in module. > > Bug: v8:9654 > Change-Id: Ia25103ca29963afa103c124ff5f159f197c2b2b0 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1970470 > Commit-Queue: Z Nguyen-Huu <duongn@microsoft.com> > Reviewed-by: Clemens Backes <clemensb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#65529} Change-Id: I035c867c99219bbf8d4fa25b58c734306e25fa63 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985032 Commit-Queue: Z Nguyen-Huu <duongn@microsoft.com> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#65614}
-
Joshua Litt authored
Fixes clusterfuzz bug. This is a reland of 15ec4a09 Original change's description: > [promises] Port Promise.race to Torque. > > Bug: v8:9838 > Change-Id: Iee3bcaa3a7149309c01d16be67d189ccc56bd0e8 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1965919 > Commit-Queue: Joshua Litt <joshualitt@chromium.org> > Reviewed-by: Tobias Tebbi <tebbi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#65562} Bug: v8:9838 Change-Id: Id295a12023195511289d92517936733ab22cdf4b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1988542Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Joshua Litt <joshualitt@chromium.org> Cr-Commit-Position: refs/heads/master@{#65613}
-
Leszek Swirski authored
PerformSideEffectCheckAtBytecode calls Isolate::TerminateExecution on failure, which can allocate. We can avoid Handles by only accessing the frame's function object after the side effect check. Bug: v8:9991 Change-Id: Iac74f8cf5ff0840a18c59faab4c256a3fa9d5b25 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1989825 Auto-Submit: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Reviewed-by: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#65612}
-
Clemens Backes authored
Instead of having both a templatized and a parameter-based method, just have the parameter-based one, but make it constexpr. DCHECKs in these constexpr methods need to be guarded because GCC 5 does not support them yet. R=thibaudm@chromium.org Bug: v8:10021 Change-Id: If362de42f1a4c2cda4435fce63beb9244795d008 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1986001Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#65611}
-
Seth Brenith authored
This change updates GetObjectProperties to list all of the bitfields within a class field, if that class field's type is a bitfield struct. The representation of bitfields in the GetObjectProperties response is very similar to the representation of struct fields, but with two extra bytes of data specifying the shift and size of the bitfield. Bug: v8:9376 Change-Id: I40a22169f3d01652a7f2db8cface43c2a1e30cfe Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1960835Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Seth Brenith <seth.brenith@microsoft.com> Cr-Commit-Position: refs/heads/master@{#65610}
-
Michael Lippautz authored
Change-Id: Ifc3100595c88a374ac41cb0d90825c23c1cae756 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1980704 Auto-Submit: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#65609}
-
Tobias Tebbi authored
This reverts commit a3039123. Reason for revert: https://crrev.com/c/1985990 should fix the root-cause now. Original change's description: > [regalloc] temporarily disable control-flow aware allocation > > Bug: v8:10077 > > Change-Id: I4b76d62462658dfa54f351ba10e52e57f63f3493 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1977857 > Commit-Queue: Tobias Tebbi <tebbi@chromium.org> > Reviewed-by: Thibaud Michaud <thibaudm@chromium.org> > Cr-Commit-Position: refs/heads/master@{#65536} TBR=tebbi@chromium.org,thibaudm@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:10077 Change-Id: I356f1cc8d70f1f9efdbc72e3398cc830c51b664d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1986002Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#65608}
-
Joshua Litt authored
Currently, the implementation of Call throws if the property is non-callable. This CL just updates the torque type signature to reflect this. Change-Id: I3f8626ba96f41e6d52564df8d481fba52b05a3fc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1989572 Commit-Queue: Joshua Litt <joshualitt@chromium.org> Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#65607}
-
Thibaud Michaud authored
{LoadFromConstantsTable} uses r7 as a temporary register. This can conflict with the load destination register, so use another register in this case. R=jgruber@chromium.org CC=tebbi@chromium.org Bug: v8:10077 Change-Id: I3359f31ab4b17838464ee9bf0828308161fb9c0d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985990 Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#65606}
-
Dan Elphick authored
Converts StoreObjectField, StoreMapNoWriteBarrier and StoreObjectFieldRoot to use TNode parameters and convert all of their uses. Bug: v8:10021 Change-Id: Ic0511e43677f6aaaeac20df58a69be74156d390f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1980581Reviewed-by: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Mythri Alle <mythria@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#65605}
-
Dan Elphick authored
Makes the Node* versions private so they can still be called from the inlined TNode versions. Also changes to LoadFromParentFrame to return TNode<Object> since its uses don't require anything looser. Bug: v8:10021 Change-Id: I84e3831d444a7787e0b03ff2f9e665181a9caa06 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1980578 Commit-Queue: Dan Elphick <delphick@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#65604}
-
Leszek Swirski authored
LogWriteDebugInfo can allocate when calculating line ends for source positions, so make its called, LogRecordedBuffer, take Handles rather than raw Objects. This also improves its API, as we can change the maybe-null SharedFunctionInfo argument into a MaybeHandle. Bug: chromium:1037872 Change-Id: Ifa3e2d9be7aa7de3b05e5c1e107406004b8963c7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985995 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#65603}
-
Leszek Swirski authored
During conflict detection, we want to early exit the scope loop when we find a non-conflict, but continue looking at the other declarations in the scope. Bug: chromium:1038588 Change-Id: Ia2a19b02222fbd13cec70d3a60d2f5bae4ce245b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985991 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#65602}
-
Anna Henningsen authored
The error messages previously gave the impression that a cast in the sense of a JS type conversion may have been attempted. Clarify the error messages to a) indicate that the type mismatched (rather than using the term “conversion”) and b) always use the C++ class name to clarify that the check refers to the actual class hierarchy, rather than conceptual types. Change-Id: I28c1b8a265f3d96f344bc2e00ac24bd9b9b6d269 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1987247Reviewed-by: Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#65601}
-
Huáng Jùnliàng authored
When "await" is referenced in a part of an async arrow function head, we will walk up the active ambiguous expression stack and mark the ambiguous arrow head scope. However the class member initializer is always non-ambiguous so we should reset the stack and stop walking up. Bug: v8:10094 Change-Id: Iac35889158ef46c851888993b6081f8ff8610c43 Fixed: v8:10094 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1987620 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#65600}
-
Tobias Tebbi authored
Bug: chromium:1031909 Change-Id: Ibf120d722a8cb6eb9b9eaa15163cb7846dab64ea Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1981507Reviewed-by: Michael Stanton <mvstanton@chromium.org> Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#65599}
-
Maya Lekova authored
The current value of 200 is fine with the TypeScript run time (less than 8s) as long as the limit is checked at an earlier stage, which was overlooked in the initial implementation. Fixed: chromium:1038292 Change-Id: Ia512e709a79450eed631f03129ddbbff65fd81b1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985992Reviewed-by: Michael Stanton <mvstanton@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#65598}
-
Zhao Jiazhong authored
port 9dfe6359 https://crrev.com/c/1829220 [mips][builtins] Tweak optimized check in InterpreterEntryTrampoline port 7177d87f https://crrev.com/c/1833683 Bug: v8:9771 Change-Id: I1d5214fd6d6b5bf3f6c30003c616054ce4e4ee15 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1892471Reviewed-by: Bill Budge <bbudge@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#65597}
-
Clemens Backes authored
They are generated by running d8 with the '--perf-prof' flag. We never want to commit them. R=machenbach@chromium.org No-Try: true Change-Id: I1e249142b893b4f4a9c09cc75eb6623680be12f7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985994Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#65596}
-
Thibaud Michaud authored
Set the wasm engine in the {PipelineData} constructor used for JS to Wasm wrapper compilation, so that the code tracer can be accessed independently from the isolate. R=mvstanton@chromium.org Bug: chromium:1032677 Change-Id: Id26d1695446251e310fe7dbd9cc7b04f8f1ad175 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1973738Reviewed-by: Michael Stanton <mvstanton@chromium.org> Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#65595}
-
Dan Elphick authored
Converts NodeGenerator and SelectImpl to return TNode<T>. Also removes redundant PromiseBuiltinsAssembler::SetForwardingHandlerIfTrue overload. Bug: v8:10021 Change-Id: I65e23fb208e4d9bf041e0f9c9b2ea113aeac58a5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1980576Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#65594}
-
Sigurd Schneider authored
This CL improves whitespace precision of coverage around try blocks; previously a small portion of whitespace could be reported as uncovered between try blocks and catch and/or finally blocks. Change-Id: I763ae3d15106c88f2278cf8893c12b0869a62528 Fixed: v8:10030 Bug: v8:10030 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1962265Reviewed-by: Toon Verwaest <verwaest@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Cr-Commit-Position: refs/heads/master@{#65593}
-
Simon Zünd authored
This CL introduces a CHECK to narrow down the cause for a crash happening in the wild, but that currently has no reproduction case. Bug: chromium:1035651 Change-Id: I61c0658d4b753f2c7d12aa8e20d5a2ff69594d1d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985989Reviewed-by: Peter Marshall <petermarshall@chromium.org> Commit-Queue: Simon Zünd <szuend@chromium.org> Cr-Commit-Position: refs/heads/master@{#65592}
-
Pierre Langlois authored
Change-Id: I1cdd6e7f761d7fa75fd4ca5331ca2475a4384c30 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985988 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#65591}
-
v8-ci-autoroll-builder authored
Rolling v8/base/trace_event/common: https://chromium.googlesource.com/chromium/src/base/trace_event/common/+log/5e4fce1..81c050f Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/9e12430..8ddd521 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/d3a10a0..3543506 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/cf50f77..a814502 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: Ia9999ac71e4d89673216ed63ae8f0353563c1076 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1988074Reviewed-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@{#65590}
-
- 06 Jan, 2020 6 commits
-
-
Ng Zhi An authored
Bug: v8:9886 Change-Id: I711bc0c8d1a9625eb3b2ecd38a44fb201aef153e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1985031 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#65589}
-
Ng Zhi An authored
The AVX version should only take one argument, so these instructions have to be split from the main list of SSE4 instructions, whose AVX version have two arguments. Bug: v8:9886 Change-Id: Ie37e060711babd7760547e2aa01c9c0fb0c728b5 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1986215 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#65588}
-
Joshua Litt authored
This reverts commit 15ec4a09. Reason for revert: Clusterfuzz Bug:chromium:1038140 Original change's description: > [promises] Port Promise.race to Torque. > > Bug: v8:9838 > Change-Id: Iee3bcaa3a7149309c01d16be67d189ccc56bd0e8 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1965919 > Commit-Queue: Joshua Litt <joshualitt@chromium.org> > Reviewed-by: Tobias Tebbi <tebbi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#65562} TBR=tebbi@chromium.org,joshualitt@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:9838 Change-Id: Ic9aa558425f106b0c1c7f5b87dfb6064605ee657 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1988539Reviewed-by: Joshua Litt <joshualitt@chromium.org> Commit-Queue: Joshua Litt <joshualitt@chromium.org> Cr-Commit-Position: refs/heads/master@{#65587}
-
Milad Farazmand authored
Change-Id: I2b75df9c6189007d8b3aace631a8b403663500a3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1970251Reviewed-by: Junliang Yan <jyan@ca.ibm.com> Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com> Cr-Commit-Position: refs/heads/master@{#65586}
-
Mythri Alle authored
This reverts commit a4db68d4. Reason for revert: Speculatively reverting: failure on win32 bot https://ci.chromium.org/p/v8/builders/ci/V8%20Win32%20-%20debug/23495 Not sure if the test is flaky or this cl is actually making it flaky. Reverting while I investigate this issue. Original change's description: > Reland "[TurboFan] Don't serialize read-only heap objects" > > Reland the cl with fixes to TSAN failures. > > This reverts commit 03c9de73. > > > Original change's description: > > Revert "[TurboFan] Don't serialize read-only heap objects" > > > > This reverts commit 9f18e55f. > > > > Reason for revert: https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20TSAN/29660 > > > > Original change's description: > > > [TurboFan] Don't serialize read-only heap objects > > > > > > Read-only heap objects are immutable and immovable. It is safe to access > > > these objects directly from the heap. Not having to serialize them > > > reduces the time we spend on main thread especially for TurboProp. > > > > > > Bug: v8:9684 > > > Change-Id: Ibabb7076af50c9007d2a8ed57fe257406958fb6a > > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1955596 > > > Reviewed-by: Michael Stanton <mvstanton@chromium.org> > > > Reviewed-by: Maya Lekova <mslekova@chromium.org> > > > Commit-Queue: Mythri Alle <mythria@chromium.org> > > > Cr-Commit-Position: refs/heads/master@{#65490} > > > > TBR=mvstanton@chromium.org,neis@chromium.org,mythria@chromium.org,mslekova@chromium.org > > > > Change-Id: If2d8649cdc083f7d064684352501320a96a1ba2c > > No-Presubmit: true > > No-Tree-Checks: true > > No-Try: true > > Bug: v8:9684 > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1973732 > > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> > > Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#65492} > > TBR=mvstanton@chromium.org,neis@chromium.org,mythria@chromium.org,mslekova@chromium.org,nicohartmann@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: v8:9684 > Change-Id: I54c9973d0f4d88504b2fb920feb3480fe1f1e1fc > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1980573 > Reviewed-by: Mythri Alle <mythria@chromium.org> > Reviewed-by: Michael Stanton <mvstanton@chromium.org> > Commit-Queue: Mythri Alle <mythria@chromium.org> > Cr-Commit-Position: refs/heads/master@{#65584} TBR=mvstanton@chromium.org,neis@chromium.org,mythria@chromium.org,mslekova@chromium.org,nicohartmann@chromium.org Change-Id: Ic28774a2530bb7b6688a974f2af4a23ec04ef236 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:9684 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1987250Reviewed-by: Mythri Alle <mythria@chromium.org> Commit-Queue: Mythri Alle <mythria@chromium.org> Cr-Commit-Position: refs/heads/master@{#65585}
-
Mythri A authored
Reland the cl with fixes to TSAN failures. This reverts commit 03c9de73. Original change's description: > Revert "[TurboFan] Don't serialize read-only heap objects" > > This reverts commit 9f18e55f. > > Reason for revert: https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20TSAN/29660 > > Original change's description: > > [TurboFan] Don't serialize read-only heap objects > > > > Read-only heap objects are immutable and immovable. It is safe to access > > these objects directly from the heap. Not having to serialize them > > reduces the time we spend on main thread especially for TurboProp. > > > > Bug: v8:9684 > > Change-Id: Ibabb7076af50c9007d2a8ed57fe257406958fb6a > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1955596 > > Reviewed-by: Michael Stanton <mvstanton@chromium.org> > > Reviewed-by: Maya Lekova <mslekova@chromium.org> > > Commit-Queue: Mythri Alle <mythria@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#65490} > > TBR=mvstanton@chromium.org,neis@chromium.org,mythria@chromium.org,mslekova@chromium.org > > Change-Id: If2d8649cdc083f7d064684352501320a96a1ba2c > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: v8:9684 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1973732 > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> > Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> > Cr-Commit-Position: refs/heads/master@{#65492} TBR=mvstanton@chromium.org,neis@chromium.org,mythria@chromium.org,mslekova@chromium.org,nicohartmann@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: v8:9684 Change-Id: I54c9973d0f4d88504b2fb920feb3480fe1f1e1fc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1980573Reviewed-by: Mythri Alle <mythria@chromium.org> Reviewed-by: Michael Stanton <mvstanton@chromium.org> Commit-Queue: Mythri Alle <mythria@chromium.org> Cr-Commit-Position: refs/heads/master@{#65584}
-
- 05 Jan, 2020 2 commits
-
-
Ng Zhi An authored
Change-Id: I4258a218e9b2bbba5b9d14eccc9c6779e701e793 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1986248 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#65583}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/41f432e..9e12430 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/5f98f04..d3a10a0 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/4413434..cf50f77 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I56744193cc1cbcb5a511b448d066864f90e6ad63 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1984856Reviewed-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@{#65582}
-
- 03 Jan, 2020 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/19d4aa5..41f432e Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/7c4300c..5f98f04 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: Iddb93c75f9b6ad4b9cb444ce149231020e82f328 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1984852Reviewed-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@{#65581}
-
- 02 Jan, 2020 1 commit
-
-
Shu-yu Guo authored
The receiver may be undefined when calling optionally chained properties, so CallAnyReceiver should be used instead of CallProperty. TBR=rmcilroy@chromium.org Bug: chromium:1038178 Change-Id: Id91f2ecda1a5b38f6d1c9a6b6f90c0ae7dcbe638 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1986205 Commit-Queue: Shu-yu Guo <syg@chromium.org> Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#65580}
-