- 13 Feb, 2015 1 commit
-
-
Sven Panne authored
This removed some inconsistent/duplicated code and made the testing side entrance much simpler: Now it simply constructs a pipeline and hands off to the "tail" of the normal pipeline. R=danno@chromium.org Review URL: https://codereview.chromium.org/924823002 Cr-Commit-Position: refs/heads/master@{#26637}
-
- 12 Feb, 2015 2 commits
-
-
svenpanne authored
Use a fake code stub instead, basically following the null object pattern. Review URL: https://codereview.chromium.org/918973002 Cr-Commit-Position: refs/heads/master@{#26610}
-
mstarzinger authored
R=danno@chromium.org TEST=cctest/test-run-intrinsics/CallFunction Review URL: https://codereview.chromium.org/916873002 Cr-Commit-Position: refs/heads/master@{#26604}
-
- 11 Feb, 2015 6 commits
-
-
mstarzinger authored
R=bmeurer@chromium.org TEST=unittests/CommonOperatorTest Review URL: https://codereview.chromium.org/912393002 Cr-Commit-Position: refs/heads/master@{#26584}
-
svenpanne authored
A CompilationInfo constructed from just an Isolate* and a Zone* is in weird an inconsistent state (calling e.g. flags() on it will crash), so we need to avoid them. This CL removes almost all of them, the remaining 2 call sites in (for testing only) will be handled in a separate CL. Things which have been changed: * Linkage is basically a decorator for CallDescriptor now. * ChangeLowering doesn't need Linkage at all. * JSGenericLowering doesn't need a full CompilationInfo*, just a single flag. * JSContextSpecializer doesn't need the full CompilationInfo, just a Context. * Removed unused CompilationInfo from SimplifiedLoweringTester. This nicely decouples things already a bit more, but there's still work to do... Review URL: https://codereview.chromium.org/899803003 Cr-Commit-Position: refs/heads/master@{#26580}
-
titzer authored
BUG= Review URL: https://codereview.chromium.org/898353002 Cr-Commit-Position: refs/heads/master@{#26576}
-
dslomov authored
R=arv@chromium.org,rossberg@chromium.org BUG=v8:3834 LOG=N Committed: https://crrev.com/8aed43e82c6d2742fe5988603cb8841324cc942b Cr-Commit-Position: refs/heads/master@{#26560} Review URL: https://codereview.chromium.org/908883002 Cr-Commit-Position: refs/heads/master@{#26572}
-
dslomov authored
Revert of new classes: implement new.target passing to superclass constructor. (patchset #9 id:150001 of https://codereview.chromium.org/908883002/) Reason for revert: Breaks Linux64 release Original issue's description: > new classes: implement new.target passing to superclass constructor. > > R=arv@chromium.org,rossberg@chromium.org > BUG=v8:3834 > LOG=N > > Committed: https://crrev.com/8aed43e82c6d2742fe5988603cb8841324cc942b > Cr-Commit-Position: refs/heads/master@{#26560} TBR=arv@chromium.org,rossberg@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3834 Review URL: https://codereview.chromium.org/911363002 Cr-Commit-Position: refs/heads/master@{#26561}
-
dslomov authored
R=arv@chromium.org,rossberg@chromium.org BUG=v8:3834 LOG=N Review URL: https://codereview.chromium.org/908883002 Cr-Commit-Position: refs/heads/master@{#26560}
-
- 06 Feb, 2015 1 commit
-
-
dcarney authored
Review URL: https://codereview.chromium.org/904693002 Cr-Commit-Position: refs/heads/master@{#26479}
-
- 05 Feb, 2015 1 commit
-
-
danno authored
Make sure the initial graph is fully populated with source position information and automatically propagate that information down through newly allocated nodes during reduction passes in the most unobtrusive way that's currently possible. Review URL: https://codereview.chromium.org/897883002 Cr-Commit-Position: refs/heads/master@{#26459}
-
- 03 Feb, 2015 2 commits
-
-
Benedikt Meurer authored
If a (pure) node has two or more uses, but there exists a path from the common dominator of these uses to end, which does not contain a use, then we split the node such that no unnecessary computation takes place. Note however, that this only applies if the node cannot be hoisted out of a loop. BUG=v8:3864 LOG=n R=jarin@chromium.org Review URL: https://codereview.chromium.org/899433005 Cr-Commit-Position: refs/heads/master@{#26404}
-
danno authored
R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/886313006 Cr-Commit-Position: refs/heads/master@{#26399}
-
- 30 Jan, 2015 3 commits
-
-
bmeurer authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/877753007 Cr-Commit-Position: refs/heads/master@{#26346}
-
Benedikt Meurer authored
This reverts commit 6a4c0a3b and commit 0deaa4b6 for breaking GCC bots. TBR=svenpanne@chromium.org Review URL: https://codereview.chromium.org/893533003 Cr-Commit-Position: refs/heads/master@{#26342}
-
bmeurer authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/888613002 Cr-Commit-Position: refs/heads/master@{#26340}
-
- 29 Jan, 2015 2 commits
-
-
danno authored
Review URL: https://codereview.chromium.org/882973002 Cr-Commit-Position: refs/heads/master@{#26323}
-
titzer authored
R=jarin@chromium.org BUG= Review URL: https://codereview.chromium.org/877553007 Cr-Commit-Position: refs/heads/master@{#26318}
-
- 26 Jan, 2015 1 commit
-
-
bmeurer authored
The lowering of intrinsics is therefore now decoupled from the general inlining logic. TEST=cctest,unittests R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/872363002 Cr-Commit-Position: refs/heads/master@{#26263}
-
- 23 Jan, 2015 1 commit
-
-
danno authored
Along the way: - Thread isolate parameter explicitly through code that used to rely on getting it from the zone. - Canonicalize the parameter position of isolate and zone for affected code - Change Hydrogen New<> instruction templates to automatically pass isolate R=mstarzinger@chromium.org LOG=N Review URL: https://codereview.chromium.org/868883002 Cr-Commit-Position: refs/heads/master@{#26252}
-
- 22 Jan, 2015 1 commit
-
-
mstarzinger authored
R=titzer@chromium.org TEST=mjsunit/regress/regress-crbug-451016 BUG=chromium:451016 LOG=N Review URL: https://codereview.chromium.org/817293005 Cr-Commit-Position: refs/heads/master@{#26229}
-
- 20 Jan, 2015 1 commit
-
-
titzer authored
BUG= Review URL: https://codereview.chromium.org/816053002 Cr-Commit-Position: refs/heads/master@{#26149}
-
- 19 Jan, 2015 1 commit
-
-
mstarzinger authored
R=rossberg@chromium.org Review URL: https://codereview.chromium.org/855873002 Cr-Commit-Position: refs/heads/master@{#26131}
-
- 16 Jan, 2015 2 commits
-
-
mstarzinger authored
R=rossberg@chromium.org Review URL: https://codereview.chromium.org/826833004 Cr-Commit-Position: refs/heads/master@{#26104}
-
mstarzinger authored
R=rossberg@chromium.org,jarin@chromium.org TEST=cctest/test-run-jsops/ClassLiteral Review URL: https://codereview.chromium.org/798873006 Cr-Commit-Position: refs/heads/master@{#26101}
-
- 15 Jan, 2015 1 commit
-
-
dcarney authored
BUG= Review URL: https://codereview.chromium.org/798873008 Cr-Commit-Position: refs/heads/master@{#26074}
-
- 14 Jan, 2015 1 commit
-
-
mstarzinger authored
R=rossberg@chromium.org TEST=cctest/test-run-jsbranches/ForOfContinueStatement Review URL: https://codereview.chromium.org/850803002 Cr-Commit-Position: refs/heads/master@{#26060}
-
- 12 Jan, 2015 1 commit
-
-
titzer authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/809333002 Cr-Commit-Position: refs/heads/master@{#26020}
-
- 08 Jan, 2015 1 commit
-
-
bmeurer authored
- Use C++11 range based for loops. - Remove duplicated virtual register set in unittests. - Don't expose implementation details of InstructionSelector. TEST=unittests R=dcarney@chromium.org Review URL: https://codereview.chromium.org/837423002 Cr-Commit-Position: refs/heads/master@{#25997}
-
- 07 Jan, 2015 2 commits
-
-
dcarney authored
BUG= Review URL: https://codereview.chromium.org/837173002 Cr-Commit-Position: refs/heads/master@{#25970}
-
dcarney authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/810403003 Cr-Commit-Position: refs/heads/master@{#25968}
-
- 05 Jan, 2015 1 commit
-
-
Benedikt Meurer authored
Introduce a new AnyToBoolean simplified operator to handle the later lowering of boolean conversions. Previously we tried to hack that with the generic JSToBoolean, having its context set to zero, but that lead to various problems/bugs and did not handle all cases. TEST=cctest,unittests R=jarin@chromium.org Review URL: https://codereview.chromium.org/800833003 Cr-Commit-Position: refs/heads/master@{#25958}
-
- 22 Dec, 2014 1 commit
-
-
Benedikt Meurer authored
The CommonOperatorReducer currently takes care of redundant Phis, EffectPhis and Selects. This functionality overlaps with ControlReducer, but is required to make certain optimizations effective, since the ControlReducer only runs really early and really late in the pipeline and therefore other reducers aren't reapplied properly after redundant phi/select elimination. TEST=unittests R=hpayer@chromium.org Review URL: https://codereview.chromium.org/817243003 Cr-Commit-Position: refs/heads/master@{#25922}
-
- 18 Dec, 2014 1 commit
-
-
Benedikt Meurer authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/809313002 Cr-Commit-Position: refs/heads/master@{#25874}
-
- 16 Dec, 2014 1 commit
-
-
Benedikt Meurer authored
This greatly reduces the number of nodes in the graph (by more than 20x in some extreme cases) for the Emscripten python interpreter main function. BUG=v8:3763 LOG=y TEST=cctest,mjsunit,unittests R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/802353003 Cr-Commit-Position: refs/heads/master@{#25840}
-
- 12 Dec, 2014 5 commits
-
-
https://codereview.chromium.org/803493002/dcarney authored
Reason for revert: performance bots were unchanged by the original revert Original issue's description: > revert r25736 > > R=bmeurer@chromium.org > > BUG= TBR=bmeurer@chromium.org NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/787183003 Cr-Commit-Position: refs/heads/master@{#25801}
-
dcarney authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/803493002 Cr-Commit-Position: refs/heads/master@{#25795}
-
dcarney authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/800493002 Cr-Commit-Position: refs/heads/master@{#25794}
-
svenpanne authored
FINAL implies OVERRIDE, which in turn implies virtual, so there's no need to use more than one of these. The Google C++ style guide even requires this, see http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Inheritance. While we're here, port r24662 to x87. The net result is that v8 compiles again with a current clang. BUG=v8:3753 LOG=y Review URL: https://codereview.chromium.org/797943002 Cr-Commit-Position: refs/heads/master@{#25792}
-
Benedikt Meurer authored
- Decouple JSBuiltinReducer from JSTypedLowering. - Unify JSTypedLowering::ReduceJSToXXX() lowering. - Cleanup several includes and forward declarations. - Unify helper methods. TEST=cctest R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/797903003 Cr-Commit-Position: refs/heads/master@{#25790}
-