- 12 Jul, 2016 8 commits
-
-
machenbach authored
Temporary commit to see performance data. Will be reverted shortly after. BUG=chromium:622332 TBR=vogelheim, jochen NOTRY=true Review-Url: https://codereview.chromium.org/2135313002 Cr-Commit-Position: refs/heads/master@{#37667}
-
yangguo authored
R=jgruber@chromium.org Review-Url: https://codereview.chromium.org/2139613002 Cr-Commit-Position: refs/heads/master@{#37666}
-
bmeurer authored
Remove obsolete definitions from macros.py, and drop the now obsolete %_ToPrimitive, %_ToPrimitive_Number, %_ToPrimitive_String, %_ToName and the %ToPrimitive_String intrinsics/runtime entries. R=yangguo@chromium.org BUG=v8:5049 Review-Url: https://codereview.chromium.org/2137203002 Cr-Commit-Position: refs/heads/master@{#37665}
-
yangguo authored
R=franzih@chromium.org BUG=chromium:515897 Review-Url: https://codereview.chromium.org/2137033002 Cr-Commit-Position: refs/heads/master@{#37664}
-
diaoyuanjie authored
Added trace printout for replacements in Turbofan reducers. Renamed graph trimmer trace flag to avoid confusion. Review-Url: https://codereview.chromium.org/2123283006 Cr-Commit-Position: refs/heads/master@{#37663}
-
v8-autoroll authored
Rolling v8/build to 3cd1ead0173f28d3aa6b28590495ae67340a3a5e Rolling v8/third_party/icu to b5ecbb29a26532f72ef482569b223d5a51fd50bf Rolling v8/tools/mb to 9a768a4d908ec89978a95e1c04c62336f5cfbcd2 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2144483002 Cr-Commit-Position: refs/heads/master@{#37662}
-
bbudge authored
AllocateSpillSlot can now handle requests for 16 byte slots. LOG=N BUG=v8:4124 Review-Url: https://codereview.chromium.org/2139663002 Cr-Commit-Position: refs/heads/master@{#37661}
-
alph authored
BUG=v8:4789 Committed: https://crrev.com/c3a16f0a9f0976411ee25df627534be5f6955c32 Review-Url: https://codereview.chromium.org/2133533002 Cr-Original-Commit-Position: refs/heads/master@{#37618} Cr-Commit-Position: refs/heads/master@{#37660}
-
- 11 Jul, 2016 32 commits
-
-
bjaideep authored
Port d49d3864 R=jgruber@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG=v8:5173 LOG=N Review-Url: https://codereview.chromium.org/2141723003 Cr-Commit-Position: refs/heads/master@{#37659}
-
diaoyuanjie authored
Graph trimmer can set Input as null and subsequent printing call could cause segmentation fault. R=bmeurer@chromium.org, jarin@chromium.org Review-Url: https://codereview.chromium.org/2134443002 Cr-Commit-Position: refs/heads/master@{#37658}
-
jochen authored
Instead just return undefined A side effect of this is that it's no longer possible to compile functions in a detached context. BUG=chromium:541703 R=verwaest@chromium.org,bmeurer@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng Review-Url: https://codereview.chromium.org/2034083002 Cr-Commit-Position: refs/heads/master@{#37657}
-
jwolfe authored
When the scanner finds a '-->', it's either part of an HTMLCloseComment or a '--' followed by a '>'. Previously, only a preceding newline would make it an HTMLCloseComment. Now, a preceding multiline comment also makes it an HTMLCloseComment. The effect is that now the following is not a SyntaxError: x/* */-->this is now a comment BUG=v8:5142 LOG=y Review-Url: https://codereview.chromium.org/2119763003 Cr-Commit-Position: refs/heads/master@{#37656}
-
littledan authored
This patch just checks for a stack overflow and returns failure from the cases which Clusterfuzz found. However, there may be more locations in the parser which need similar treatment. R=caitpotter88@gmail.com,neis BUG=v8:4483,chromium:624300 Review-Url: https://codereview.chromium.org/2135503002 Cr-Commit-Position: refs/heads/master@{#37655}
-
bbudge authored
LOG=N BUG=v8:4124 Review-Url: https://codereview.chromium.org/2139513002 Cr-Commit-Position: refs/heads/master@{#37654}
-
aseemgarg authored
BUG=613928 R=bradnelson@chromium.org TEST=regress-613928.js Review-Url: https://codereview.chromium.org/2123283007 Cr-Commit-Position: refs/heads/master@{#37653}
-
bmeurer authored
This is follow-up cleanup for the flags that are no longer used inside SimplifiedLowering. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2132403002 Cr-Commit-Position: refs/heads/master@{#37652}
-
ishell authored
[runtime] Follow-up fix for "Better encapsulation of dictionary objects handling in lookup iterator." BUG=chromium:626715 Review-Url: https://codereview.chromium.org/2135253002 Cr-Commit-Position: refs/heads/master@{#37651}
-
bbudge authored
Clean up call sites. LOG=N BUG=v8:4124 Review-Url: https://codereview.chromium.org/2124983004 Cr-Commit-Position: refs/heads/master@{#37650}
-
bbudge authored
All architectures return false for now. LOG=N BUG=v8:4124 Review-Url: https://codereview.chromium.org/2135573003 Cr-Commit-Position: refs/heads/master@{#37649}
-
hpayer authored
BUG= Review-Url: https://codereview.chromium.org/2138783002 Cr-Commit-Position: refs/heads/master@{#37648}
-
jochen authored
R=verwaest@chromium.org,dcheng@chromium.org BUG= Review-Url: https://codereview.chromium.org/2137943003 Cr-Commit-Position: refs/heads/master@{#37647}
-
mstarzinger authored
This removes the frame state input representing the before-state from nodes having any comparison operator. Lowering that inserts number conversions of the inputs has to be disabled when deoptimization is enabled, because the frame state layout is no longer known. R=jarin@chromium.org BUG=v8:5021 Review-Url: https://codereview.chromium.org/2134173002 Cr-Commit-Position: refs/heads/master@{#37646}
-
mstarzinger authored
This extends pretenuring decisions based on allocation sites to heap numbers that are allocated as part of object literals. It ensures memory locality of a bigger enclosure of the deep copy of an object literal. R=bmeurer@chromium.org TEST=cctest/test-heap/OptimizedPretenuringMixedInObjectProperties Review-Url: https://codereview.chromium.org/2135933002 Cr-Commit-Position: refs/heads/master@{#37645}
-
jgruber authored
Builtin frames can simply use the existing JavaScriptFrame::Print method. Builtin exit frames need their own implementation which can print the function name, receiver and parameters. R=bmeurer@chromium.org, yangguo@chromium.org BUG= Review-Url: https://codereview.chromium.org/2134093002 Cr-Commit-Position: refs/heads/master@{#37644}
-
bmeurer authored
If the first input to CheckBounds is already an Unsigned32, then we can just truncate both inputs to word32 and lower the bounds check. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2138753002 Cr-Commit-Position: refs/heads/master@{#37643}
-
titzer authored
R=ahaas@chromium.org BUG= Review-Url: https://codereview.chromium.org/2135693002 Cr-Commit-Position: refs/heads/master@{#37642}
-
titzer authored
R=jarin@chromium.org, mstarzinger@chromium.org BUG= Review-Url: https://codereview.chromium.org/2132393002 Cr-Commit-Position: refs/heads/master@{#37641}
-
peterssen authored
BUG=chromium:508898 Review-Url: https://codereview.chromium.org/2135153002 Cr-Commit-Position: refs/heads/master@{#37640}
-
bmeurer authored
This unifies phases in SimplifiedLowering, and will enable more optimizations, i.e. we can do the output representation during typing then, which in turn should allow us to use the feedback type even in unchecked operators (i.e. NumberAbs). R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2137003002 Cr-Commit-Position: refs/heads/master@{#37639}
-
bmeurer authored
We can just do the Int32Abs lowering always for Signed32 inputs. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2134143002 Cr-Commit-Position: refs/heads/master@{#37638}
-
rmcilroy authored
BUG=v8:4280 Review-Url: https://codereview.chromium.org/2128233002 Cr-Commit-Position: refs/heads/master@{#37637}
-
neis authored
R=mstarzinger@chromium.org BUG= Review-Url: https://codereview.chromium.org/2134163002 Cr-Commit-Position: refs/heads/master@{#37636}
-
mstarzinger authored
R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2132373002 Cr-Commit-Position: refs/heads/master@{#37635}
-
caitp authored
BUG=v8:5174, v8:1543 R=cbruni@chromium.org, littledan@chromium.org Review-Url: https://codereview.chromium.org/2129193003 Cr-Commit-Position: refs/heads/master@{#37634}
-
danno authored
By adding MachineType to LinkageLocation, it is possible not only to reason about the location of a LinkageLocation on the stack, but also about it's size. This will be useful in follow-on CLs that attempt to merge some of the parameter passing logic of tail calls and normal (non-tail) calls. As a nice side-effect, it is no longer necessary to separately keep a MachineSignature in a CallDescriptor, because the MachineTypes contianed in LinkageLocation for all of the Descriptor's parameters and return types are sufficient. This CL therefore removes the MachineSignature from the CallDescriptor and adjusts all the calling code accordingly, simplifying and de-duplicating code in a bunch of places. R=titzer@chromium.org, bmeurer@chromium.org LOG=N Review-Url: https://codereview.chromium.org/2124023003 Cr-Commit-Position: refs/heads/master@{#37633}
-
jkummerow authored
Revert of Avoid Context::Enter and Context::Exit if we're reentering the active and last entered context (patchset #2 id:20001 of https://codereview.chromium.org/2131483002/ ) Reason for revert: crashes: crbug.com/626719 Original issue's description: > Avoid Context::Enter and Context::Exit if we're reentering the active and last entered context > > A bit of browsing around indicates that the new fast-path is taken most of the time: > 3496 Entering new > 152295 Reentering same > > BUG= > > Committed: https://crrev.com/1829eb661ef1cf432fb551bcb193f521a219d490 > Cr-Commit-Position: refs/heads/master@{#37570} TBR=jochen@chromium.org,verwaest@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG= Review-Url: https://codereview.chromium.org/2140563002 Cr-Commit-Position: refs/heads/master@{#37632}
-
jgruber authored
CallSite::IsConstructor() was unable to recognize builtin construct stubs (NumberConstructor_ConstructStub and StringConstructor_ConstructStub) as constructors, and thus these frames were not formatted correctly in stack traces. Fix this by explicitly marking their Code objects as construct stubs and passing along a special receiver value when we encounter such cases in CaptureSimpleStackTrace. R=mstarzinger@chromium.org, yangguo@chromium.org BUG= Review-Url: https://codereview.chromium.org/2125163004 Cr-Commit-Position: refs/heads/master@{#37631}
-
bmeurer authored
If a Smi field load is truncated to Word32, we can just load the upper 32 bits on 64-bit architectures (with 32-bit Smis), which avoids the nasty shift right that we'd have to perform otherwise to untag it. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2140553002 Cr-Commit-Position: refs/heads/master@{#37630}
-
bmeurer authored
Usually loops run from 0 to some array length l, which means the induction variable i will probably have type Unsigned32, just like the length l. The CheckBounds operation lowers to an Uint32LessThan comparison, so if we also lower the user level i < l comparison to Uint32LessThan (whenever possible), we get some bounds check elimination for free (via value numbering plus branch condition elimination). This merges the branch condition elimination phase with the late optimization phase to make this magic happen. R=jarin@chromium.org BUG=v8:4930,v8:5141 Review-Url: https://codereview.chromium.org/2135123002 Cr-Commit-Position: refs/heads/master@{#37629}
-
mstarzinger authored
This enables tests which verify allocation site feedback is used and influences pretenuring decisions. By now TurboFan is respecting such feedback. Ignition however doesn't provide such feedback yet. R=mvstanton@chromium.org Review-Url: https://codereview.chromium.org/2135563003 Cr-Commit-Position: refs/heads/master@{#37628}
-