- 21 May, 2015 6 commits
-
-
svenpanne authored
Currently the stub simply calls out to the runtime, this will be improved in a later CLs. The current state at least avoids bit-rot and later merging horror. Fixes frame construction logic for stubs, too, and contains quite a few tiny cleanups in stub-land. Review URL: https://codereview.chromium.org/1150673002 Cr-Commit-Position: refs/heads/master@{#28555}
-
bmeurer authored
The inliner previously assumed that there will only be returns reaching the end node, but that's not true. This refactoring will make it possible to also hook up Deoptimize, Throw and Terminate nodes reaching end properly. R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1146393002 Cr-Commit-Position: refs/heads/master@{#28550}
-
bmeurer authored
Replace the --turbo-deoptimization flag with --turbo-asm-deoptimization and enable deoptimization for non-asm.js TurboFan code unconditionally. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1153483002 Cr-Commit-Position: refs/heads/master@{#28543}
-
titzer authored
BUG= Review URL: https://codereview.chromium.org/1149563004 Cr-Commit-Position: refs/heads/master@{#28540}
-
jarin authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/1148133002 Cr-Commit-Position: refs/heads/master@{#28536}
-
arv authored
This allows you to put iterables into your array literals and the will get spread into the array. let x = [0, ...range(1, 3)]; // [0, 1, 2] This is done by treating the array literal up to the first spread element as usual, including using a boiler plate array, and then appending the remaining expressions and rest expressions. BUG=v8:3018 LOG=N Review URL: https://codereview.chromium.org/1125183008 Cr-Commit-Position: refs/heads/master@{#28534}
-
- 20 May, 2015 7 commits
-
-
bmeurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1145143002 Cr-Commit-Position: refs/heads/master@{#28524}
-
danno authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1144133003 Cr-Commit-Position: refs/heads/master@{#28519}
-
bmeurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1148033002 Cr-Commit-Position: refs/heads/master@{#28515}
-
danno authored
Review URL: https://codereview.chromium.org/1140743004 Cr-Commit-Position: refs/heads/master@{#28510}
-
jacob.bramley authored
`mvn xd, xn` is an architectural alias for `orn xd, xzr, xm`, so this doesn't change code generation at all, but it's tidier and it addresses a TODO. BUG= Review URL: https://codereview.chromium.org/1151483002 Cr-Commit-Position: refs/heads/master@{#28507}
-
bmeurer authored
R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1147973002 Cr-Commit-Position: refs/heads/master@{#28505}
-
erikcorry authored
R=verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/1143133002 Cr-Commit-Position: refs/heads/master@{#28502}
-
- 19 May, 2015 8 commits
-
-
mstarzinger authored
R=titzer@chromium.org TEST=mjsunit/regress/regress-crbug-489293 BUG=chromium:489293 LOG=n Review URL: https://codereview.chromium.org/1142873005 Cr-Commit-Position: refs/heads/master@{#28486}
-
mstarzinger authored
This in turn allows usage of AdvancedReducer::ReplaceWithValue which has access to the underlying graph reducer. It will allow us to deal with exception continuations correctly. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1134663006 Cr-Commit-Position: refs/heads/master@{#28485}
-
wingo authored
This reapplies https://codereview.chromium.org/1136073002, along with the followups: Remove Scope::scope_uses_this_ flag https://codereview.chromium.org/1128963005 and PPC: Resolve references to "this" the same way as normal variables https://codereview.chromium.org/1134073003 R=rossberg@chromium.org LOG=N BUG= Review URL: https://codereview.chromium.org/1136883006 Cr-Commit-Position: refs/heads/master@{#28458} Review URL: https://codereview.chromium.org/1140633003 Cr-Commit-Position: refs/heads/master@{#28484}
-
wingo authored
Revert of Reapply "Resolve references to "this" the same way as normal variables"" (patchset #2 id:20001 of https://codereview.chromium.org/1136883006/) Reason for revert: Something is deserializing "this" declarations as Variable::NORMAL and not Variable::THIS https://codereview.chromium.org/1136123010/ Original issue's description: > Reapply "Resolve references to "this" the same way as normal variables"" > > This reapplies https://codereview.chromium.org/1136073002, along with > the followups: > > Remove Scope::scope_uses_this_ flag > https://codereview.chromium.org/1128963005 > > and > > PPC: Resolve references to "this" the same way as normal variables > https://codereview.chromium.org/1134073003 > > R=yangguo@chromium.org, rossberg@chromium.org > LOG=N > BUG= > > Committed: https://crrev.com/1efc1e4f7a3d30d5225e9d5cb2585cad7cb17099 > Cr-Commit-Position: refs/heads/master@{#28458} TBR=rossberg@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1146733002 Cr-Commit-Position: refs/heads/master@{#28473}
-
balazs.kilvady authored
MIPS64r6 and MIPS32r6 support for min and max and Float64Max machine operators. BUG= Review URL: https://codereview.chromium.org/1027193002 Cr-Commit-Position: refs/heads/master@{#28471}
-
mstarzinger authored
This in turn allows usage of AdvancedReducer::ReplaceWithValue which has access to the underlying graph reducer. It will allow us to deal with exception continuations correctly. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1134303003 Cr-Commit-Position: refs/heads/master@{#28468}
-
bmeurer authored
During inlining we should pay attention to only rewire the outer frame states of the inlinee, but leave any inner frame states of the inlinee untouched. Otherwise we might run into trouble once we start caching graphs, or do getter/setter inlining. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1134973006 Cr-Commit-Position: refs/heads/master@{#28466}
-
titzer authored
R=mvstanton@chromium.org BUG= Review URL: https://codereview.chromium.org/1133113004 Cr-Commit-Position: refs/heads/master@{#28465}
-
- 18 May, 2015 3 commits
-
-
wingo authored
This reapplies https://codereview.chromium.org/1136073002, along with the followups: Remove Scope::scope_uses_this_ flag https://codereview.chromium.org/1128963005 and PPC: Resolve references to "this" the same way as normal variables https://codereview.chromium.org/1134073003 R=yangguo@chromium.org, rossberg@chromium.org LOG=N BUG= Review URL: https://codereview.chromium.org/1136883006 Cr-Commit-Position: refs/heads/master@{#28458}
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1132353004 Cr-Commit-Position: refs/heads/master@{#28452}
-
titzer authored
R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/1136413002 Cr-Commit-Position: refs/heads/master@{#28441}
-
- 15 May, 2015 3 commits
-
-
mvstanton authored
Now that vector ics are established for load, keyed load and call ics, let's remove dead code behind the flag. BUG= Review URL: https://codereview.chromium.org/1129853002 Cr-Commit-Position: refs/heads/master@{#28422}
-
bmeurer authored
First step towards support for inlining based on SharedFunctionInfo instead of JSFunction. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1134713004 Cr-Commit-Position: refs/heads/master@{#28419}
-
martyn.capewell authored
Enable clang's shorten-64-to-32 warning flag on ARM64, and fix the warnings that arise. BUG= Review URL: https://codereview.chromium.org/1131573006 Cr-Commit-Position: refs/heads/master@{#28412}
-
- 13 May, 2015 7 commits
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1109763002 Cr-Commit-Position: refs/heads/master@{#28399}
-
titzer authored
R=jarin@chromium.org BUG= Review URL: https://codereview.chromium.org/1140943003 Cr-Commit-Position: refs/heads/master@{#28398}
-
yangguo authored
... and the following two "PPC: Resolve references to "this" the same way as normal variables" "Remove Scope::scope_uses_this_ flag" R=hablich@chromium.org BUG=chromium:487289 LOG=N Review URL: https://codereview.chromium.org/1134003003 Cr-Commit-Position: refs/heads/master@{#28395}
-
bmeurer authored
If the typer was able to prove that a NumberAdd/Sub/Mul/Div/Mod always produces a Signed32/Unsigned32 value, and the inputs are Signed32/Unsigned32, we can lower the node to the corresponding integer operation instead, no matter what the uses are. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1140933002 Cr-Commit-Position: refs/heads/master@{#28393}
-
titzer authored
R=jarin@chromium.org BUG= Review URL: https://codereview.chromium.org/1133303006 Cr-Commit-Position: refs/heads/master@{#28392}
-
bmeurer authored
This was already done for other binary operations, so it's basically copying the existing functionality to shift left and shift right logical/arithmetic. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1140883003 Cr-Commit-Position: refs/heads/master@{#28389}
-
bmeurer authored
Use these check points to optimize comparisons where we already know that one side cannot be a String (or turn into a string via ToPrimitive). Also remove bunch of useless DoNotCrash tests for the scheduler that are painful to maintain and add almost no value. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1140583004 Cr-Commit-Position: refs/heads/master@{#28383}
-
- 12 May, 2015 4 commits
-
-
conradw authored
Implements the strong mode proposal's restrictions on implicit conversions for the binary >, >=, <, and <= operators. BUG=v8:3956 LOG=N Review URL: https://codereview.chromium.org/1130283002 Cr-Commit-Position: refs/heads/master@{#28370}
-
titzer authored
Handle missing or improper FrameStates more gracefully. Uses the operator properties to determine if the node contains a FrameState before, then checks if a valid bailout id exists. R=jarin@chromium.org BUG= Review URL: https://codereview.chromium.org/1135243002 Cr-Commit-Position: refs/heads/master@{#28364}
-
titzer authored
[turbofan] Add AdvancedReducer::ReplaceWithValue() method and convert JSInlining to an AdvancedReducer. Note that this is just a duplication for now. We'll want to get rid of the NodeProperties::ReplaceWithValue() method in the long run. R=bmeurer@chromium.org BUG= Review URL: https://codereview.chromium.org/1135483004 Cr-Commit-Position: refs/heads/master@{#28363}
-
titzer authored
[turbofan] Use FrameStatesBeforeAndAfter to simplify handling of before/after frame states in AstGraphBuilder. R=jarin@chromium.org BUG= Review URL: https://codereview.chromium.org/1128193005 Cr-Commit-Position: refs/heads/master@{#28361}
-
- 11 May, 2015 2 commits
-
-
arv authored
In strong mode it is an error to call a function with too few arguments. This is enforced inside the ArgumentsAdaptorTrampoline. This does not yet handle rest parameters BUG=v8:3956 LOG=N R=rossberg@chromium.org, dslomov@chromium.org Review URL: https://codereview.chromium.org/1115263004 Cr-Commit-Position: refs/heads/master@{#28346}
-
wingo authored
R=rossberg@chromium.org LOG=N BUG= Review URL: https://codereview.chromium.org/1128963005 Cr-Commit-Position: refs/heads/master@{#28343}
-