- 26 Nov, 2018 18 commits
-
-
Toon Verwaest authored
Change-Id: Ib584fa4fa38d1f042dc7220f516e35c0619ace45 Reviewed-on: https://chromium-review.googlesource.com/c/1350127Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#57827}
-
Michael Starzinger authored
R=ulan@chromium.org BUG=v8:6792,v8:8157 Change-Id: Idf43b4dc74ad7ba89142f2480ed8632adf75f8ec Reviewed-on: https://chromium-review.googlesource.com/c/1348069 Commit-Queue: Michael Starzinger <mstarzinger@chromium.org> Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#57826}
-
Aleksandar Rikalo authored
Addition to d0562944. Bug: chromium:896326 Change-Id: Ie76f6be778e963ad5205a99853f491b164dddf9a Reviewed-on: https://chromium-review.googlesource.com/c/1349269Reviewed-by:
Predrag Rudic <prudic@wavecomp.com> Reviewed-by:
Sreten Kovacevic <skovacevic@wavecomp.com> Commit-Queue: Sreten Kovacevic <skovacevic@wavecomp.com> Cr-Commit-Position: refs/heads/master@{#57825}
-
Hannes Payer authored
Bug: chromium:852420 Change-Id: Ie0a6eeb42b57db2309403acd23e986f73cb33afe Reviewed-on: https://chromium-review.googlesource.com/c/1350123 Commit-Queue: Hannes Payer <hpayer@chromium.org> Reviewed-by:
-
Jakob Gruber authored
The ProfileEntryHookStub is no longer used and can be removed. Bug: v8:7777, v8:8503 Change-Id: I4ccd75d38cfee3e7963338d5d8213915db9be4a9 Reviewed-on: https://chromium-review.googlesource.com/c/1349191 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Michael Starzinger <mstarzinger@chromium.org> Reviewed-by:
Yang Guo <yangguo@chromium.org> Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/master@{#57823}
-
Sigurd Schneider authored
This script counts lines of code before and after preprocessor expansion. When running: tools/locs.py --build-dir out.gn/repro --smallest --largest --worst in the v8 directory, it reports: Processed 1,520 files in 24.62 sec. src ( 625 files): 365,679 to 53,468,383 ( 146.22x) third_party ( 432 files): 239,081 to 9,502,094 ( 39.74x) gen ( 19 files): 64,040 to 1,415,287 ( 22.10x) total ( 1520 files): 1,173,349 to 102,831,949 ( 87.64x) test ( 392 files): 489,865 to 37,476,036 ( 76.50x) Largest 3 files after expansion: 33,495 to 140,401 ( 4.19x) gen/torque-generated/builtins-array-from-dsl-gen.cc 100,014 to 140,111 ( 1.40x) ../../test/cctest/gay-precision.cc 100,014 to 140,110 ( 1.40x) ../../test/cctest/gay-shortest.cc Worst expansion (3 files): 6 to 100,826 (14403.71x) ../../test/fuzzer/wasm-data-section.cc 6 to 100,826 (14403.71x) ../../test/fuzzer/wasm-function-sigs-section.cc 6 to 100,826 (14403.71x) ../../test/fuzzer/wasm-globals-section.cc Smallest 3 input files: 0 to 0 ( 0.00x) ../../testing/gtest/empty.cc 1 to 0 ( 0.00x) ../../src/x64/simulator-x64.cc 4 to 4,423 ( 884.60x) ../../buildtools/third_party/libc++/trunk/src/utility.cpp Notry: true Change-Id: Ie1da942463d9e6a84338a9bd57d9ddc46d2856dc Reviewed-on: https://chromium-review.googlesource.com/c/1349571 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#57822}
-
Mike Stanton authored
R=jkummerow@chromium.org Bug: v8:8238 Change-Id: Ie28326ebe6c69e194857aed7b5d49cb8e5a40a29 Reviewed-on: https://chromium-review.googlesource.com/c/1349243Reviewed-by:
-
Jakob Gruber authored
It's been failing for a while, and looks like we reach some arbitrary heap limit, triggering an early OOM before the debugger hook is set. Bug: v8:8494 Change-Id: I472dc8955ba2f0eb018ac6e7ca83e4beaaefc318 Reviewed-on: https://chromium-review.googlesource.com/c/1350830Reviewed-by:
Sigurd Schneider <sigurds@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#57820}
-
Michael Starzinger authored
R=clemensh@chromium.org Change-Id: I325b10268a9ed9548fd28ecc3e5e2b0959afcdc4 Reviewed-on: https://chromium-review.googlesource.com/c/1350125Reviewed-by:
Clemens Hammacher <clemensh@chromium.org> Commit-Queue: Michael Starzinger <mstarzinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#57819}
-
Toon Verwaest authored
No-Tree-Checks: true No-Try: true Change-Id: I8b8651e499c2d862ae9f81a18af11d610214c0cd Reviewed-on: https://chromium-review.googlesource.com/c/1350118 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#57818}
-
Toon Verwaest authored
No-Tree-Checks: true No-Try: true Change-Id: If9e39eb64db24e7d7dd2ae639d95f750aef04210 Reviewed-on: https://chromium-review.googlesource.com/c/1350119 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
Toon Verwaest authored
No-Tree-Checks: true No-Try: true Change-Id: I88fb20bb78df733fb135d2103b38908926a9d512 Reviewed-on: https://chromium-review.googlesource.com/c/1350115 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
Toon Verwaest authored
No-Tree-Checks: true No-Try: true Bug: chromium:908231 Change-Id: I1acf33400ad3546974a0ccb5955f8c6966b17dcf Reviewed-on: https://chromium-review.googlesource.com/c/1350116 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
Marja Hölttä authored
- Remove heap-inl.h includes from places where it looked unnecessary. (This is a non-scientific approach, because it's probably pulled in indirectly anyway.) - Annotate places which include heap-inl.h because they need heap/ internals. - ACCESSORS legitimately needs heap-inl.h because of Heap::FromWritableHeapObject. - Add includes to heap/heap-write-barrier(-inl).h - A bunch of IWYU fixes discovered when working on this CL (includes which were missing because heap-inl.h pulls them in indirectly). BUG=v8:7490,v8:8238,v8:8499 Change-Id: I00f9a74d430f13d7c080dca77a92b03bcca7ef96 Reviewed-on: https://chromium-review.googlesource.com/c/1349241Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
Toon Verwaest authored
Bug: chromium:908250 Change-Id: I414255f00bf7dcec0cf8b35e314a2b9b78a9116e Reviewed-on: https://chromium-review.googlesource.com/c/1350117 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
Hannes Payer authored
Bug: chromium:852420 Change-Id: I106b783899cef2f124450f8cc266b3b2ac09b4ae Reviewed-on: https://chromium-review.googlesource.com/c/1350114Reviewed-by:
-
Maya Lekova authored
This reverts commit ca086a49. Reason for revert: Seems to be the cause for https://bugs.chromium.org/p/chromium/issues/detail?id=908296 Original change's description: > [parser] Perfect hash for keywords > > Use gperf to generate a perfect hash table for keyword lookup. Adds a > python script which munges the output of gperf and adds additional > cleanup and optimisations. > > Change-Id: I3656a7287dbd0688917893de3a671faef9e4578a > Reviewed-on: https://chromium-review.googlesource.com/c/1349240 > Commit-Queue: Leszek Swirski <leszeks@chromium.org> > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Reviewed-by: Marja Hölttä <marja@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57790} TBR=marja@chromium.org,leszeks@chromium.org,verwaest@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: Ia0cc283f21e9f6793522c46a1fd40ba2d88597fb Reviewed-on: https://chromium-review.googlesource.com/c/1350113Reviewed-by:
Maya Lekova <mslekova@chromium.org> Commit-Queue: Maya Lekova <mslekova@chromium.org> Cr-Commit-Position: refs/heads/master@{#57811}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/b471e77..2339062 TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: Id6d0f397d17a6602222e109de565b41ff7451d0b Reviewed-on: https://chromium-review.googlesource.com/c/1350284 Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#57810}
-
- 25 Nov, 2018 3 commits
-
-
Jakob Kummerow authored
Bug: v8:3770 Change-Id: Id515906744a738d5d40dbb6dee15e243623f020c Reviewed-on: https://chromium-review.googlesource.com/c/1349111 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Peter Marshall <petermarshall@chromium.org> Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#57809}
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/fa87ce6..b471e77 TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: Ie8d912df3b6c2cdb4688fca4070e58734ed412f1 Reviewed-on: https://chromium-review.googlesource.com/c/1350269Reviewed-by:
-
Jakob Kummerow authored
Removing the temporarily duplicated classes FixedArrayPtr and FixedArrayBasePtr. Bug: v8:3770 Change-Id: I056ad74ff69593e9f134ef5c976766812c4d9275 Reviewed-on: https://chromium-review.googlesource.com/c/1345913 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Ben Titzer <titzer@chromium.org> Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
- 24 Nov, 2018 8 commits
-
-
Sigurd Schneider authored
This reverts commit 81b5f713. Revert "[heap] Cleanup embedder tracing APIs" This reverts commit ce02d86b. Tbr: mlippautz@chromium.org Change-Id: I5900ac3c070c93b869c9173316a466d39287713a Reviewed-on: https://chromium-review.googlesource.com/c/1350111Reviewed-by:
-
Clemens Hammacher authored
This is a reland of e8a22a70 Original change's description: > [iwyu] Add missing includes > > This fixes a presubmit error: > https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Presubmit/454 > > It also adds another missing include. > > R=machenbach@chromium.org > CC=caitp@igalia.com > > Bug: v8:7490 > Change-Id: I915a5bf431073410a67ff08471fd5e6974e31aa1 > No-Try: true > No-Tree-Checks: true > Reviewed-on: https://chromium-review.googlesource.com/c/1346111 > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Commit-Queue: Clemens Hammacher <clemensh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57671} TBR=clemensh@chromium.org NOTRY=true NOTREECHECKS=true Bug: v8:7490 Change-Id: Id327a36cc5a727e734ea1bb6faf43e592ee2f21c Reviewed-on: https://chromium-review.googlesource.com/c/1350129Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#57805}
-
Jakob Kummerow authored
Bug: v8:3770 Change-Id: If5328a4c63d8efe0ce7a0c5a744666c79c02e1ee Reviewed-on: https://chromium-review.googlesource.com/c/1345912 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Hannes Payer <hpayer@chromium.org> Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
-
Jakob Kummerow authored
Bug: v8:3770 Change-Id: I49d4fdc1cac6c4bde81fbe0bf76341be12711109 Reviewed-on: https://chromium-review.googlesource.com/c/1345911 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/cd7b727..fa87ce6 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/4ed4737..17079a5 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/25c4fce..6c18a1a TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: I2b67323a005e4dea8f3fe5bd36dd5b94c4d4f744 Reviewed-on: https://chromium-review.googlesource.com/c/1349732Reviewed-by:
-
Jakob Kummerow authored
to the new design. Bug: v8:3770 Change-Id: I3cd0a66eefefedc98a641494302fc79d897a153a Reviewed-on: https://chromium-review.googlesource.com/c/1345910 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
Jaroslav Sevcik <jarin@chromium.org> Reviewed-by:
-
Jakob Kummerow authored
Bug: v8:3770 Change-Id: I06f7fb1b2915d1c87162cb464d0ed34d08516e24 Reviewed-on: https://chromium-review.googlesource.com/c/1345909 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
-
Jakob Kummerow authored
Bug: v8:3770 Change-Id: I0bac3cc2a0a2c7e6c1cf18e7db00bdffd2d9f7a1 Reviewed-on: https://chromium-review.googlesource.com/c/1345328 Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
-
- 23 Nov, 2018 11 commits
-
-
Caitlin Potter authored
This is a reland of a5336471 Original change's description: > [builtins] Implement Object.fromEntries > > Adds the Object.fromEntries() method behind > --harmony-object-from-entries. > > > Includes an initial implementation of the new experimental builtin > Object.fromEntries implemented by Daniel Clifford, and > has been modified by Caitlin Potter to support a fast case to skip > the iterator protocol when it can be done unobservably in common cases. > > There are some incidental changes: A number of CSA macros have been > updated to use TNodes, and some Context arguments have been > re-arranged to be implicit in Torque. > > > There are also a number of mjsunit tests written mirroring and > expanding on the test262 tests. > > BUG=v8:8021 > > Change-Id: I1c12bee8a2f98c6297b77d5d723910a5e3b630cc > Co-authored-by: Daniel Clifford <danno@chromium.org> > Co-authored-by: Caitlin Potter <caitp@igalia.com> > Reviewed-on: https://chromium-review.googlesource.com/c/1337585 > Commit-Queue: Daniel Clifford <danno@chromium.org> > Reviewed-by: Daniel Clifford <danno@chromium.org> > Reviewed-by: Tobias Tebbi <tebbi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57667} Bug: v8:8021 Change-Id: I706e2d87bfc2f688e833c1b7d40ca82f5d80f5a2 Reviewed-on: https://chromium-review.googlesource.com/c/1346630Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Caitlin Potter <caitp@igalia.com> Cr-Commit-Position: refs/heads/master@{#57798}
-
Toon Verwaest authored
This reverts commit 8c359f64. Reason for revert: Tentative revert for ASAN breakage Original change's description: > [zone] Keep one page when we Zone::Reset for reuse > > Change-Id: I50c6124d3da5b35d4156c066f38d10d2dc966567 > Reviewed-on: https://chromium-review.googlesource.com/c/1349246 > Reviewed-by: Igor Sheludko <ishell@chromium.org> > Commit-Queue: Toon Verwaest <verwaest@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57793} TBR=ishell@chromium.org,verwaest@chromium.org Change-Id: Iff319b5f0ad1a65b1171d219a02a4a75d33910bd No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/1349248Reviewed-by:
-
Toon Verwaest authored
Change-Id: I78dfc1ae0ec7bc47c14d2d525d2f20516e13effa Reviewed-on: https://chromium-review.googlesource.com/c/1349247Reviewed-by:
-
Michael Lippautz authored
Provide processing scope that makes it impossible to maintain locally cached wrappers that could get invalidated in Blink and yield in crashers. Bug: chromium:843903, v8:8238 Change-Id: I7ba1905f6c77a97bcc61ac42f921dcac4772471f Reviewed-on: https://chromium-review.googlesource.com/c/1349276 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
Toon Verwaest authored
Change-Id: Icbda182a894ce6508efbfa3bdb17ba3adce360c7 Reviewed-on: https://chromium-review.googlesource.com/c/1349573Reviewed-by:
-
Toon Verwaest authored
Change-Id: I50c6124d3da5b35d4156c066f38d10d2dc966567 Reviewed-on: https://chromium-review.googlesource.com/c/1349246Reviewed-by:
-
Toon Verwaest authored
Rather check expressions used as patterns directly. Check parentheses by tagging parenthesized expressions as parenthesized. This allows us to drop UnexpectedPatternToken and makes it clear why a specific token is unexpected (because it's invalid in a binding pattern). This also more uniformly restores messages like "Invalid destructuring assignment target". Change-Id: Idd98e9116c85de4c2304cf1fef1baa097b67149d Reviewed-on: https://chromium-review.googlesource.com/c/1349572 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
Hannes Payer authored
Bug: chromium:852420 Change-Id: I348dbf52bc43078861e40e8377257c8f9a2bc920 Reviewed-on: https://chromium-review.googlesource.com/c/1349242Reviewed-by:
-
Leszek Swirski authored
Use gperf to generate a perfect hash table for keyword lookup. Adds a python script which munges the output of gperf and adds additional cleanup and optimisations. Change-Id: I3656a7287dbd0688917893de3a671faef9e4578a Reviewed-on: https://chromium-review.googlesource.com/c/1349240 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
-
Peter Marshall authored
- Remove the InstallFunction variant which just passed its arguments straight to JSObject::AddProperty - Change InstallFunction to accept a String rather than a Symbol as all symbol installation goes through InstallFunctionAtSymbol now. This way we can avoid the call to Name::ToFunctionName as well - Add an explicit helper InstallFunctionWithBuiltinId for installing functions which have a builtin ID. These are always installed with DONT_ENUM PropertyAttributes so we can remove that parameter, too. - Remove PropertyAttributes from InstallFunction because it is always DONT_ENUM. Bug: v8:8238 Change-Id: I7af3d6d833d50065c20e198e21a72ef4a539c1ca Reviewed-on: https://chromium-review.googlesource.com/c/1349284Reviewed-by:
-
Andreas Haas authored
R=yangguo@chromium.org Bug: v8:8238 Change-Id: Ia59aefc54c2e9f4fa3348c42fb45e7fadab8ee76 Reviewed-on: https://chromium-review.googlesource.com/c/1349231Reviewed-by:
-
