- 08 Dec, 2014 8 commits
-
-
dcarney authored
R=jarin@chromium.org BUG= Review URL: https://codereview.chromium.org/777403003 Cr-Commit-Position: refs/heads/master@{#25704}
-
yangguo authored
R=jochen@chromium.org Review URL: https://codereview.chromium.org/790433002 Cr-Commit-Position: refs/heads/master@{#25703}
-
Jochen Eisinger authored
BUG=none TBR=rossberg@chromium.org LOG=n Review URL: https://codereview.chromium.org/788603002 Cr-Commit-Position: refs/heads/master@{#25702}
-
jochen authored
When compiling with the macro DCHECK_ALWAYS_ON defined, DCHECKs and supporting code gets compiled and enabled. This increases test coverage for chromium release buildbots BUG=v8:3731 R=jkummerow@chromium.org LOG=y Review URL: https://codereview.chromium.org/760213005 Cr-Commit-Position: refs/heads/master@{#25701}
-
machenbach authored
BUG=chromium:374740 LOG=n TEST=python -m unittest run_perf_test NOTRY=true TBR=ulan@chromium.org Review URL: https://codereview.chromium.org/776223003 Cr-Commit-Position: refs/heads/master@{#25700}
-
Benedikt Meurer authored
TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/783913003 Cr-Commit-Position: refs/heads/master@{#25699}
-
Benedikt Meurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/786703002 Cr-Commit-Position: refs/heads/master@{#25698}
-
adamk authored
Review URL: https://codereview.chromium.org/780323002 Cr-Commit-Position: refs/heads/master@{#25697}
-
- 05 Dec, 2014 25 commits
-
-
adamk authored
Revert of Optimize add/set/delete operations for string keys in Maps and Sets (patchset #1 id:1 of https://codereview.chromium.org/777663003/) Reason for revert: Deopt fuzzer reports that something's still broken, reverting until I can create a minimal repro. Original issue's description: > Optimize add/set/delete operations for string keys in Maps and Sets > > This was previously landed in commit 66e2f60b, but failed the > collections mjsunit test with --deopt-every-n=1 due to a typo > in the shrinking code. This patch corrects and simplifies the > shrinking logic, and the tests now pass. > > R=dslomov@chromium.org > > Committed: https://chromium.googlesource.com/v8/v8/+/8599f5f047b4400f7f61de9e449b2246c9aad471 TBR=dslomov@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/742353006 Cr-Commit-Position: refs/heads/master@{#25696}
-
Adam Klein authored
This was previously landed in commit 66e2f60b, but failed the collections mjsunit test with --deopt-every-n=1 due to a typo in the shrinking code. This patch corrects and simplifies the shrinking logic, and the tests now pass. R=dslomov@chromium.org Review URL: https://codereview.chromium.org/777663003 Cr-Commit-Position: refs/heads/master@{#25695}
-
arv authored
This introduces Hydrogen for %_GetPrototype. The code falls back on runtime if the object needs access checks or if its prototype is a hidden prototype. BUG=None LOG=Y R=dslomov@chromium.org Review URL: https://codereview.chromium.org/756423006 Cr-Commit-Position: refs/heads/master@{#25694}
-
paul.lind authored
Port 35ff259b. BUG= Review URL: https://codereview.chromium.org/756383005 Cr-Commit-Position: refs/heads/master@{#25693}
-
balazs.kilvady authored
Port 35ff259b Original commit message: This combines Map::DoneInobjectSlackTracking and Map::ConstructionCount into one more generic 4-bit counter. Counter values from 15 down to 8 are used for in-object slack tracking, values from 7 down to 0 are free to be used for a new counter when in-object slack tracking is inactive BUG= Review URL: https://codereview.chromium.org/780303003 Cr-Commit-Position: refs/heads/master@{#25692}
-
baixo authored
BUG=421063 Review URL: https://codereview.chromium.org/749213004 Cr-Commit-Position: refs/heads/master@{#25691}
-
dslomov authored
R=rossberg@chromium.org Review URL: https://codereview.chromium.org/779203005 Cr-Commit-Position: refs/heads/master@{#25690}
-
ulan authored
This combines Map::DoneInobjectSlackTracking and Map::ConstructionCount into one more generic 4-bit counter. Counter values from 15 down to 8 are used for in-object slack tracking, values from 7 down to 0 are free to be used for a new counter when in-object slack tracking is inactive. Review URL: https://codereview.chromium.org/767253002 Cr-Commit-Position: refs/heads/master@{#25689}
-
dcarney authored
R=svenpanne@chromium.org BUG= Review URL: https://codereview.chromium.org/765883003 Cr-Commit-Position: refs/heads/master@{#25688}
-
ulan authored
This fixes mjsunit/es6/mirror-collections failure in arm64 gc-stress bot. BUG= Review URL: https://codereview.chromium.org/783673002 Cr-Commit-Position: refs/heads/master@{#25687}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/781953002 Cr-Commit-Position: refs/heads/master@{#25686}
-
dcarney authored
R=titzer@chromium.org, BUG= Review URL: https://codereview.chromium.org/781933002 Cr-Commit-Position: refs/heads/master@{#25685}
-
dslomov authored
R=jarin@chromium.org BUG=v8:3741 LOG=N Review URL: https://codereview.chromium.org/762393008 Cr-Commit-Position: refs/heads/master@{#25684}
-
danno authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/771073003 Cr-Commit-Position: refs/heads/master@{#25683}
-
mstarzinger authored
R=dcarney@chromium.org Review URL: https://codereview.chromium.org/778093003 Cr-Commit-Position: refs/heads/master@{#25682}
-
yangguo authored
CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg,v8_linux_nosnap_rel Review URL: https://codereview.chromium.org/781943002 Cr-Commit-Position: refs/heads/master@{#25681}
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/782703002 Cr-Commit-Position: refs/heads/master@{#25680}
-
jochen authored
Compare with LCodeGen::FinishCode BUG=none R=jarin@chromium.org,mstarzinger@chromium.org LOG=n Review URL: https://codereview.chromium.org/777243002 Cr-Commit-Position: refs/heads/master@{#25679}
-
mstarzinger authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/765833004 Cr-Commit-Position: refs/heads/master@{#25678}
-
danno authored
Achieve more than parity with modes currently handled on ia32 in preparation for porting the entire mechanism to ia32, including supporting mul of constants 3, 5 and 9 with "leal" instructions. TEST=unittests Review URL: https://codereview.chromium.org/774193003 Cr-Commit-Position: refs/heads/master@{#25677}
-
Benedikt Meurer authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/778113002 Cr-Commit-Position: refs/heads/master@{#25676}
-
jochen authored
BUG=none R=machenbach@chromium.org LOG=n Review URL: https://codereview.chromium.org/781763002 Cr-Commit-Position: refs/heads/master@{#25675}
-
machenbach authored
The new config is now used for perf measurements. The old ones are deleted to avoid redundancy. This also ports https://codereview.chromium.org/754523004 to the new config file. BUG=chromium:374740 LOG=n TBR=ulan@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/782673003 Cr-Commit-Position: refs/heads/master@{#25674}
-
Benedikt Meurer authored
This is an initial version of redundant load elimination, currently limited to LoadField operators, and implemented by walking the effect chain. TEST=unittests R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/782473002 Cr-Commit-Position: refs/heads/master@{#25673}
-
v8-autoroll authored
TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/777373002 Cr-Commit-Position: refs/heads/master@{#25672}
-
- 04 Dec, 2014 7 commits
-
-
paul.lind authored
Port c16b8f6c TEST=mjsunit/regress/regress-437765 BUG= Review URL: https://codereview.chromium.org/750213003 Cr-Commit-Position: refs/heads/master@{#25671}
-
adamk authored
Revert of Optimize add/set/delete operations for string keys in Maps and Sets (patchset #5 id:70001 of https://codereview.chromium.org/773993002/) Reason for revert: Caused test failures in mjsunit/es6/collections with --deopt-every-n-times=1 Original issue's description: > Optimize add/set/delete operations for string keys in Maps and Sets TBR=dslomov@chromium.org,arv@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/777233003 Cr-Commit-Position: refs/heads/master@{#25670}
-
arv authored
BUG=v8:3330 LOG=N TBR=dslomov@chromium.org Review URL: https://codereview.chromium.org/754523004 Cr-Commit-Position: refs/heads/master@{#25669}
-
adamk authored
Review URL: https://codereview.chromium.org/773993002 Cr-Commit-Position: refs/heads/master@{#25668}
-
arv authored
This is for performance. Having to do the test in every Advance was too expensive. BUG=438991, v8:3230 LOG=N R=dslomov@chromium.org, marja Review URL: https://codereview.chromium.org/766193003 Cr-Commit-Position: refs/heads/master@{#25667}
-
vogelheim authored
These methods for used for compressed libraries, where GetSource* functions contained the compressed sources and [GS]etRawSource* the uncompressed sources. This is dead code since the API no longer supports compression. (If you need/want compressed sources, use the external startup data and compress/uncompress on the Embedder's side.) BUG= Review URL: https://codereview.chromium.org/772853003 Cr-Commit-Position: refs/heads/master@{#25666}
-
adamk authored
This patch introduces a new inline runtime function, %_HasFastPackedElements(), and uses it both in the implementation of the 'in' operator and in the array builtins to speed up testing for the existence of an index in an array. In testing with the microbenchmark on the attached bug, for example, the runtime goes from 326ms to 66ms. A reviewer might ask whether the HAS_INDEX macro is worthwhile, and I tried the same example without it, which pushed the microbenchmark up to 157ms. So it seems it's worth it to avoid the function call to IN() if we know we're dealing with arrays and numbers. BUG=v8:3701 LOG=n Review URL: https://codereview.chromium.org/754863002 Cr-Commit-Position: refs/heads/master@{#25665}
-