- 17 Nov, 2014 10 commits
-
-
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 21 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}
-
Ben L. Titzer authored
R=verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/718393004 Cr-Commit-Position: refs/heads/master@{#25355}
-
Michael Starzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/726823002 Cr-Commit-Position: refs/heads/master@{#25354}
-
Ben L. Titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/724133004 Cr-Commit-Position: refs/heads/master@{#25353}
-
machenbach authored
Revert of Whitespace change to test CQ. (patchset #1 id:1 of https://codereview.chromium.org/725873002/) Reason for revert: Test revert. Original issue's description: > Whitespace change to test CQ. > > TBR=machenbach@chromium.org > > Committed: https://chromium.googlesource.com/v8/v8/+/42340bdfbd8b4f002257e55464f2e0b81c6ea424 TBR= NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/725893002 Cr-Commit-Position: refs/heads/master@{#25352}
-
machenbach authored
TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/725873002 Cr-Commit-Position: refs/heads/master@{#25351}
-
Dan Carney authored
BUG= R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/727693002 Cr-Commit-Position: refs/heads/master@{#25350}
-
Michael Achenbach authored
BUG=chromium:410721 LOG=n TBR=tandrii@chromium.org TEST=script_test.py Review URL: https://codereview.chromium.org/725073002 Cr-Commit-Position: refs/heads/master@{#25349}
-
Andy Wingo authored
R=mvstanton@chromium.org, svenpanne@chromium.org Review URL: https://codereview.chromium.org/670953003 Cr-Commit-Position: refs/heads/master@{#25348}
-
Benedikt Meurer authored
Drive-by-Fix: minint % 0 was broken on ARM, but we didn't notice because there was no test covering that case... TEST=msjunit R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/727673002 Cr-Commit-Position: refs/heads/master@{#25347}
-
Hannes Payer authored
BUG= R=jochen@chromium.org Review URL: https://codereview.chromium.org/725583005 Cr-Commit-Position: refs/heads/master@{#25346}
-
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 Review URL: https://codereview.chromium.org/726513002 Cr-Commit-Position: refs/heads/master@{#25345}
-
Benedikt Meurer authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/725063002 Cr-Commit-Position: refs/heads/master@{#25344}
-
- 13 Nov, 2014 7 commits
-
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#25343}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#25342}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#25341}
-
André Baixo authored
BUG=421063 LOG=N R=vogelheim@chromium.org Review URL: https://codereview.chromium.org/727513002 Patch from André Baixo <baixo@google.com>. Cr-Commit-Position: refs/heads/master@{#25340}
-
Daniel Vogelheim authored
BUG=399580,431699 LOG=N R=dcarney@chromium.org, marja@chromium.org Review URL: https://codereview.chromium.org/718043002 Cr-Commit-Position: refs/heads/master@{#25339}
-
ishell@chromium.org authored
This reverts the commit because it possibly causes renderer hangs. BUG=chromium:431807 LOG=N R=verwaest@chromium.org Review URL: https://codereview.chromium.org/725633002 Cr-Commit-Position: refs/heads/master@{#25338}
-
Erik Arvidsson authored
Now that we keep track of whether a function contains super we do not need kDefaultConstructorCallSuper BUG=None LOG=Y R=dslomov@chromium.org Review URL: https://codereview.chromium.org/718833002 Cr-Commit-Position: refs/heads/master@{#25337}
-