- 09 Apr, 2015 3 commits
-
-
jing.bao authored
BUG=v8:4015 LOG=n Review URL: https://codereview.chromium.org/1069683002 Cr-Commit-Position: refs/heads/master@{#27683}
-
hablich authored
Revert of X87: Reimplement Maps and Sets in JS (patchset #1 id:1 of https://codereview.chromium.org/1066373002/) Reason for revert: Reverting as it resulted in a closed waterfall. Original issue's description: > X87: Reimplement Maps and Sets in JS > > port 909500aa (r27605) > > original commit message: > Previously, the only optimized code path for Maps and Sets was for String keys. > This was achieved through an implementation of various complex operations > in Hydrogen. This approach was neither scalable nor forward-compatible. > > This patch adds the necessary intrinsics to implement Maps and Sets almost entirely > in JS. The added intrinsics are: > > %_FixedArrayGet > %_FixedArraySet > %_TheHole > %_JSCollectionGetTable > %_StringGetRawHashField > > With these additions, as well as a few changes to what's exposed as runtime functions, > most of the C++ code backing Maps and Sets is gone (including both runtime code in > objects.cc and Crankshaft in hydrogen.cc). > > BUG= > > Committed: https://crrev.com/56600a35a49ffa5abcba66b14839089de3589ad9 > Cr-Commit-Position: refs/heads/master@{#27681} TBR=weiliang.lin@intel.com,chunyang.dai@intel.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1073723002 Cr-Commit-Position: refs/heads/master@{#27682}
-
chunyang.dai authored
port 909500aa (r27605) original commit message: Previously, the only optimized code path for Maps and Sets was for String keys. This was achieved through an implementation of various complex operations in Hydrogen. This approach was neither scalable nor forward-compatible. This patch adds the necessary intrinsics to implement Maps and Sets almost entirely in JS. The added intrinsics are: %_FixedArrayGet %_FixedArraySet %_TheHole %_JSCollectionGetTable %_StringGetRawHashField With these additions, as well as a few changes to what's exposed as runtime functions, most of the C++ code backing Maps and Sets is gone (including both runtime code in objects.cc and Crankshaft in hydrogen.cc). BUG= Review URL: https://codereview.chromium.org/1066373002 Cr-Commit-Position: refs/heads/master@{#27681}
-
- 08 Apr, 2015 36 commits
-
-
arv authored
BUG=v8:4001 LOG=N R=adamk@chromium.org, rossberg@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1051373002 Cr-Commit-Position: refs/heads/master@{#27680}
-
balazs.kilvady authored
Port 9af9f1d0 Original commit message: These operators compute the absolute floating point value of some arbitrary input, and are implemented without any branches (i.e. using vabs on arm, and andps/andpd on x86). BUG= Review URL: https://codereview.chromium.org/1073463003 Cr-Commit-Position: refs/heads/master@{#27679}
-
dusan.milosavljevic authored
TEST= BUG= Review URL: https://codereview.chromium.org/1067393003 Cr-Commit-Position: refs/heads/master@{#27678}
-
caitpotter88 authored
Instead of "unexpected token" errors, report something meatier and more actionable. BUG= R=marja@chromium.org, arv@chromium.org LOG=N Review URL: https://codereview.chromium.org/1062263002 Cr-Commit-Position: refs/heads/master@{#27677}
-
arv authored
BUG=v8:3754 LOG=N R=dslomov@chromium.org, rossberg@chromium.org Review URL: https://codereview.chromium.org/1054883003 Cr-Commit-Position: refs/heads/master@{#27676}
-
mstarzinger authored
TBR=machenbach@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1070843002 Cr-Commit-Position: refs/heads/master@{#27675}
-
loislo authored
BUG=chromium:452067 LOG=n Committed: https://crrev.com/baf927ff5115ec62a6dad684b9232ed9d3960e3a Cr-Commit-Position: refs/heads/master@{#27626} Review URL: https://codereview.chromium.org/1045753002 Cr-Commit-Position: refs/heads/master@{#27674}
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1069763002 Cr-Commit-Position: refs/heads/master@{#27673}
-
svenpanne authored
Although all this code might be moved around later, it's a good idea to unhackify it now. :-) Review URL: https://codereview.chromium.org/1067193004 Cr-Commit-Position: refs/heads/master@{#27672}
-
mvstanton authored
If we optimize a function before gathering feedback it may be peppered with soft deoptimizations. So it can't help but deoptimize. A judicious reading of the code isn't enough to determine what the optimization state should be in the face of such chaotic gyrations. BUG= R=verwaest@chromium.org Review URL: https://codereview.chromium.org/1069363003 Cr-Commit-Position: refs/heads/master@{#27671}
-
mstarzinger authored
TBR=machenbach@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1067413002 Cr-Commit-Position: refs/heads/master@{#27670}
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1038953002 Cr-Commit-Position: refs/heads/master@{#27669}
-
titzer authored
Because, well, JavaScript is tricky and compilers like folding. R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1069673002 Cr-Commit-Position: refs/heads/master@{#27668}
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1071473003 Cr-Commit-Position: refs/heads/master@{#27667}
-
mstarzinger authored
This allows loopy TurboFan code to be interrupted by placing a stack check (i.e. JSStackCheck node) into each loop. Note that we currently limit this to non-asm.js code. Also note that stack checks are actually placed after loop headers and not at back-branches, which allows us to reuse existing BailoutIds from Crankshaft. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1065923002 Cr-Commit-Position: refs/heads/master@{#27666}
-
dusan.milosavljevic authored
TEST= BUG= Review URL: https://codereview.chromium.org/1066403002 Cr-Commit-Position: refs/heads/master@{#27665}
-
mvstanton authored
Revert of VectorICs: Turn on vector ICs for LOAD and KEYED_LOAD cases. (patchset #1 id:1 of https://codereview.chromium.org/1070653002/) Reason for revert: ARM test failure, need to investigate. Original issue's description: > VectorICs: Turn on vector ICs for LOAD and KEYED_LOAD cases. > > R=verwaest@chromium.org > BUG= > > Committed: https://crrev.com/2395eda3bb7aca938751938df76e01ac2b89b0a6 > Cr-Commit-Position: refs/heads/master@{#27657} TBR=verwaest@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1072463002 Cr-Commit-Position: refs/heads/master@{#27664}
-
mstarzinger authored
R=bmeurer@chromium.org TEST=test262-es6/language/expressions/addition/S9.3_A4.1_T2 Review URL: https://codereview.chromium.org/1071543003 Cr-Commit-Position: refs/heads/master@{#27663}
-
Benedikt Meurer authored
These operators compute the absolute floating point value of some arbitrary input, and are implemented without any branches (i.e. using vabs on arm, and andps/andpd on x86). R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/1066393002 Cr-Commit-Position: refs/heads/master@{#27662}
-
mstarzinger authored
TBR=machenbach@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1067303003 Cr-Commit-Position: refs/heads/master@{#27661}
-
marja authored
R=rossberg@chromium.org BUG= Review URL: https://codereview.chromium.org/1071533002 Cr-Commit-Position: refs/heads/master@{#27660}
-
yangguo authored
R=jarin@chromium.org BUG=chromium:474297 LOG=N Review URL: https://codereview.chromium.org/1062143002 Cr-Commit-Position: refs/heads/master@{#27659}
-
titzer authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/1061303002 Cr-Commit-Position: refs/heads/master@{#27658}
-
mvstanton authored
R=verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/1070653002 Cr-Commit-Position: refs/heads/master@{#27657}
-
yangguo authored
TBR=jkummerow@chromium.org BUG=v8:4016 LOG=N Review URL: https://codereview.chromium.org/1071513002 Cr-Commit-Position: refs/heads/master@{#27656}
-
mstarzinger authored
TBR=yangguo@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1072433002 Cr-Commit-Position: refs/heads/master@{#27655}
-
yangguo authored
Revert of Remove support for thread-based recompilation (patchset #1 id:1 of https://codereview.chromium.org/966653002/) Reason for revert: speculative revert due to gc-stress timeouts. Original issue's description: > Remove support for thread-based recompilation > > BUG=v8:3608 > R=yangguo@chromium.org > LOG=y > > Committed: https://crrev.com/ed5db223a19dfe126af012e894582251aa3635d7 > Cr-Commit-Position: refs/heads/master@{#27619} TBR=jochen@chromium.org NOPRESUBMIT=true NOTREECHECKS=true BUG=v8:3608 LOG=N Review URL: https://codereview.chromium.org/1063383004 Cr-Commit-Position: refs/heads/master@{#27654}
-
machenbach authored
NOTRY=true NOTREECHECKS=true Review URL: https://codereview.chromium.org/1064043002 Cr-Commit-Position: refs/heads/master@{#27653}
-
mstarzinger authored
TBR=yangguo@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1067353002 Cr-Commit-Position: refs/heads/master@{#27652}
-
mstarzinger authored
R=yangguo@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1070663002 Cr-Commit-Position: refs/heads/master@{#27651}
-
svenpanne authored
Perhaps we should throw an exception and/or change our external API to use a MaybeLocal, but that would be a bigger change. For now, we just return undefined when something goes wrong with the DebugContext, which is good enough to avoid crashing. BUG=chromium:474538 LOG=y Review URL: https://codereview.chromium.org/1065213002 Cr-Commit-Position: refs/heads/master@{#27650}
-
mstarzinger authored
This is a follow-up to 2d281e71 and prevents bailouts on empty script contexts in Crankshaft, which don't need allocation. Only non-empty script contexts should cause a bailout. R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/1063373003 Cr-Commit-Position: refs/heads/master@{#27649}
-
jing.bao authored
BUG=v8:4015 LOG=n Review URL: https://codereview.chromium.org/1040603002 Cr-Commit-Position: refs/heads/master@{#27648}
-
svenpanne authored
This should have been part of https://codereview.chromium.org/1056863002/. :-/ TBR=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1070593002 Cr-Commit-Position: refs/heads/master@{#27647}
-
bmeurer authored
Performance measurements show that the position independent code is usually slightly faster than the position dependent code, and there seems to be no noticable regression. This also gets rid of a lot of support code that was only required to allow embedding labels into the code stream. And it implies that neither the GC nor the deserializer need to do anything for jump tables. R=svenpanne@chromium.org BUG=v8:3872 LOG=n Review URL: https://codereview.chromium.org/1069633002 Cr-Commit-Position: refs/heads/master@{#27646}
-
v8-autoroll authored
Rolling v8/tools/clang to 7ec0c52c7a66cd94c96b8eb37bacd26d152e40e6 TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1070533002 Cr-Commit-Position: refs/heads/master@{#27645}
-
- 07 Apr, 2015 1 commit
-
-
balazs.kilvady authored
Port c67cb287 BUG= Review URL: https://codereview.chromium.org/1061363003 Cr-Commit-Position: refs/heads/master@{#27644}
-