- 18 Nov, 2014 11 commits
-
-
alexanderk authored
1) Missing <stdio.h> while using fopen, fclose, fseek etc 2) platform-linux.cc includes <sys/types.h> wtice BUG=v8:3691 Review URL: https://codereview.chromium.org/716123003 Cr-Commit-Position: refs/heads/master@{#25395}
-
Jakob Kummerow authored
The clearing mechanism relies on comparing the cached handler with the installed handler. If we try to preserve monomorphism by pretending that the IC was in uninitialized state, then it will go premonomorphic first, which means on the next miss there's no installed handler available to compare against. Instead, pretend it was premonomorphic, so the comparison will happen right away, will fail as it should, and the cached handler will be cleared. Thanks to Weiliang for starting the investigation that led to this. R=verwaest@chromium.org Review URL: https://codereview.chromium.org/730383002 Cr-Commit-Position: refs/heads/master@{#25394}
-
Michael Stanton authored
This reverts commit d40204f8. TBR=danno@chromium.org Review URL: https://codereview.chromium.org/735653003 Cr-Commit-Position: refs/heads/master@{#25393}
-
Michael Stanton authored
We were deopting without learning anything. BUG=v8:3417 LOG=N R=danno@chromium.org Review URL: https://codereview.chromium.org/368263003 Cr-Commit-Position: refs/heads/master@{#25392}
-
yangguo authored
R=ishell@chromium.org Review URL: https://codereview.chromium.org/736543003 Cr-Commit-Position: refs/heads/master@{#25391}
-
Benedikt Meurer authored
This essentially performs the following transformation JSToNumber(phi(x1,...,xn,control):primitive) => phi(JSToNumber(x1),...,JSToNumber(xn),control):number which is similar to what we already do for JSToBoolean. TEST=mjsunit/asm R=svenpanne@chromium.org BUG= Review URL: https://codereview.chromium.org/732463003 Cr-Commit-Position: refs/heads/master@{#25390}
-
aandrey authored
BUG=chromium:432468 R=yangguo@chromium.org, adamk@chromium.org, arv@chromium.org LOG=N Review URL: https://codereview.chromium.org/734163002 Cr-Commit-Position: refs/heads/master@{#25389}
-
Benedikt Meurer authored
TEST=unittests R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/735623002 Cr-Commit-Position: refs/heads/master@{#25388}
-
Michael Stanton authored
R=hpayer@chromium.org Review URL: https://codereview.chromium.org/726423002 Cr-Commit-Position: refs/heads/master@{#25387}
-
Benedikt Meurer authored
TEST=mjsunit/asm R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/728103007 Cr-Commit-Position: refs/heads/master@{#25386}
-
Benedikt Meurer authored
Also add some debug code to verify correct zero extension of 32-bit moves. TEST=mjsunit/asm R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/736623002 Cr-Commit-Position: refs/heads/master@{#25385}
-
- 17 Nov, 2014 18 commits
-
-
Adam Klein authored
R=rossberg@chromium.org BUG=v8:3076 LOG=n Review URL: https://codereview.chromium.org/733483003 Cr-Commit-Position: refs/heads/master@{#25384}
-
dslomov authored
We add a new ScopeType, ScopeType.Script. The scope with ScopeType.Script is always present in the scope chain (ScopeIterator fakes it if neededi - i.e. if ScriptContext for a script has not been allocated since that script has no lexical declarations). ScriptScope reflects ScriptContextTable. R=yurys@chromium.org,yangguo@chromium.org BUG=v8:3690 LOG=N Review URL: https://codereview.chromium.org/726643002 Cr-Commit-Position: refs/heads/master@{#25383}
-
Dan Carney authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/727373002 Cr-Commit-Position: refs/heads/master@{#25382}
-
jarin authored
BUG= Review URL: https://codereview.chromium.org/714413004 Cr-Commit-Position: refs/heads/master@{#25381}
-
Andrey Adaikin authored
R=yangguo@chromium.org, vsevik LOG=Y Committed: https://code.google.com/p/v8/source/detail?r=d5f5d38f73f43eba9658d91ffbe511af8c340d78 Review URL: https://codereview.chromium.org/710273002 Cr-Commit-Position: refs/heads/master@{#25380}
-
Dan Carney authored
BUG= R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/727323002 Cr-Commit-Position: refs/heads/master@{#25379}
-
yangguo authored
Review URL: https://codereview.chromium.org/733023003 Cr-Commit-Position: refs/heads/master@{#25378}
-
Benedikt Meurer authored
Don't use the generic algorithm, but instead start going into the direction of ControlReducer, using a stack plus a revisit queue to not miss any more possibilities for reductions anymore. TEST=cctest,unittests R=dcarney@chromium.org Committed: https://chromium.googlesource.com/v8/v8/+/f047507370634155113d78685372630a230613cf Committed: https://chromium.googlesource.com/v8/v8/+/6e148989a4227a5290a7f8ca72c71f5740870afe Review URL: https://codereview.chromium.org/726513002 Cr-Commit-Position: refs/heads/master@{#25377}
-
Benedikt Meurer authored
Now we actually implement it the way it is meant to be, that is: JSToBoolean(Phi(x1,...,xn):primitive) => Phi(JSToBoolean(x1),...,JSToBoolean(xn)):boolean This also fixes the endless recursion within JSTypedLowering when the GraphReducer does all possible reductions instead of using the generic algorithm. R=dcarney@chromium.org Review URL: https://codereview.chromium.org/730043002 Cr-Commit-Position: refs/heads/master@{#25376}
-
Ben L. Titzer authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/734593003 Cr-Commit-Position: refs/heads/master@{#25375}
-
Daniel Clifford authored
R=bmeurer@chromium.org TBR=bmeurer@chromium.org Review URL: https://codereview.chromium.org/733783003 Cr-Commit-Position: refs/heads/master@{#25374}
-
Daniel Clifford authored
Preparation for supporting more addressing modes in instruction selection. R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/729853005 Cr-Commit-Position: refs/heads/master@{#25373}
-
Michael Starzinger authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/734633002 Cr-Commit-Position: refs/heads/master@{#25372}
-
Jakob Kummerow authored
BUG=chromium:433766 LOG=n R=jarin@chromium.org Review URL: https://codereview.chromium.org/731953003 Cr-Commit-Position: refs/heads/master@{#25371}
-
Hannes Payer authored
BUG= R=jochen@chromium.org Review URL: https://codereview.chromium.org/726363002 Cr-Commit-Position: refs/heads/master@{#25370}
-
Hannes Payer authored
BUG= R=jochen@chromium.org Review URL: https://codereview.chromium.org/732473002 Cr-Commit-Position: refs/heads/master@{#25369}
-
Jaroslav Sevcik authored
With Int32Add we lose the int/uint distinction, so later, in simplified lowering we can make a wrong decision. E.g., see the attached test case, where we lower NumberAdd -> Int32Add because inputs are Uint32, but during simplified lowering we change the inputs to Int32, so we get a wrong result. Simplified lowering will lower the NumberAdd operations anyway, so we should lose performance. BUG= R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/721723004 Cr-Commit-Position: refs/heads/master@{#25368}
-
yangguo authored
Review URL: https://codereview.chromium.org/707463002 Cr-Commit-Position: refs/heads/master@{#25367}
-
- 15 Nov, 2014 2 commits
-
-
arv authored
If a class extends another class and it doesn't provide a constructor, one is created for them. We therefore need to ensure that stepping into the constructor steps into the super class constructor. BUG=v8:3674 LOG=Y R=dslomov@chromium.org, aandrey , yurys Review URL: https://codereview.chromium.org/725983002 Cr-Commit-Position: refs/heads/master@{#25366}
-
machenbach authored
BUG=chromium:433273 LOG=n NOTRY=true Review URL: https://codereview.chromium.org/728983004 Cr-Commit-Position: refs/heads/master@{#25365}
-
- 14 Nov, 2014 9 commits
-
-
adamk authored
BUG=v8:3684 LOG=n Review URL: https://codereview.chromium.org/726773002 Cr-Commit-Position: refs/heads/master@{#25364}
-
adamk authored
This requires putting the original loop's body inside an inner for loop (with the same labels as the original loop) and re-binding the temp variables in its "next" expression. A second flag is added to the desugared code to ensure the loop body executes at most once per loop. BUG=v8:3683 LOG=y Review URL: https://codereview.chromium.org/720863002 Cr-Commit-Position: refs/heads/master@{#25363}
-
caitpotter88 authored
BUG=v8:3230 Review URL: https://codereview.chromium.org/663683006 Cr-Commit-Position: refs/heads/master@{#25362}
-
dcarney authored
BUG= Review URL: https://codereview.chromium.org/727733002 Cr-Commit-Position: refs/heads/master@{#25361}
-
arv authored
Named class declarations and class expression have a const binding for the name that is in TDZ for the extends expression. BUG=v8:3330 LOG=Y R=dslomov@chromium.org, adamk Review URL: https://codereview.chromium.org/722793005 Cr-Commit-Position: refs/heads/master@{#25360}
-
Michael Starzinger authored
R=rossberg@chromium.org, titzer@chromium.org TEST=unittests/JSTypedLoweringTest.JSStrictEqualWithTheHole Review URL: https://codereview.chromium.org/722223003 Cr-Commit-Position: refs/heads/master@{#25359}
-
Dmitry Lomov authored
R=aandrey@chromium.org, rossberg@chromium.org, yurys@chromium.org BUG=v8:3690 LOG=N Review URL: https://codereview.chromium.org/732543002 Cr-Commit-Position: refs/heads/master@{#25358}
-
Andy Wingo authored
R=svenpanne@chromium.org BUG= Review URL: https://codereview.chromium.org/679943004 Cr-Commit-Position: refs/heads/master@{#25357}
-
Benedikt Meurer authored
This reverts commit 6e148989 for breaking Massive/Embenchen. TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/727743002 Cr-Commit-Position: refs/heads/master@{#25356}
-