- 18 Nov, 2014 21 commits
-
-
cullinan authored
Without this change, `make android_arm.release.check` fails since the unittests binary does not exist on the device. BUG=v8:3695 LOG= Review URL: https://codereview.chromium.org/722373003 Cr-Commit-Position: refs/heads/master@{#25405}
-
Balazs Kilvady authored
Port 47f55bae Original commit message: Code was vulnerable to different evaluation order in Clang. BUG= R=paul.lind@imgtec.com Review URL: https://codereview.chromium.org/735783002 Patch from Balazs Kilvady <balazs.kilvady@imgtec.com>. Cr-Commit-Position: refs/heads/master@{#25404}
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/742433002 Cr-Commit-Position: refs/heads/master@{#25403}
-
yangguo authored
R=dcarney@chromium.org BUG=chromium:420240 LOG=Y API=v8::String::Concat may return empty handle on overflow. Review URL: https://codereview.chromium.org/735763002 Cr-Commit-Position: refs/heads/master@{#25402}
-
mstarzinger authored
R=titzer@chromium.org Review URL: https://codereview.chromium.org/735583003 Cr-Commit-Position: refs/heads/master@{#25401}
-
yangguo authored
BUG=chromium:432493 LOG=Y Review URL: https://codereview.chromium.org/728103008 Cr-Commit-Position: refs/heads/master@{#25400}
-
mstarzinger authored
R=jarin@chromium.org TEST=cctest/test-scheduler/LoopedFloatingDiamond3 Review URL: https://codereview.chromium.org/726953002 Cr-Commit-Position: refs/heads/master@{#25399}
-
Michael Achenbach authored
BUG=chromium:374740 LOG=n TEST=tools/try_perf.py sunspider octane R=phajdan.jr@chromium.org Review URL: https://codereview.chromium.org/734403002 Cr-Commit-Position: refs/heads/master@{#25398}
-
Michael Stanton authored
Code was vulnerable to different evaluation order in Clang. R=danno@chromium.org Review URL: https://codereview.chromium.org/726693004 Cr-Commit-Position: refs/heads/master@{#25397}
-
rickyz authored
This avoids UBSan warnings about unaligned memory access. BUG= Review URL: https://codereview.chromium.org/734873002 Cr-Commit-Position: refs/heads/master@{#25396}
-
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 1 commit
-
-
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}
-