- 29 May, 2015 1 commit
-
-
v8-autoroll authored
Rolling v8/tools/clang to 5413f2a11e1b56c30e28769b9ff1086797c11b28 TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1167453002 Cr-Commit-Position: refs/heads/master@{#28690}
-
- 28 May, 2015 22 commits
-
-
arv authored
Follow up to https://codereview.chromium.org/1135243004 For now we need to call set_base_id in AstNumberingVisitor because TurboFan needs a bailout id for super.prop in a lot of cases. BUG=None LOG=N R=adamk@chromium.org, mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1153053005 Cr-Commit-Position: refs/heads/master@{#28689}
-
bbudge authored
Revert of Clean up aligned allocation code in preparation for SIMD alignments. (patchset #14 id:300001 of https://codereview.chromium.org/1150593003/) Reason for revert: Breaks mjsunit, webkit, mozilla, benchmarks. TBR=hpayer@chromium.org Original issue's description: > Clean up aligned allocation code in preparation for SIMD alignments. > > Moves alignment fill calculations into two static Heap methods. > Adds a Heap method to handle the complex case where filler is potentially needed before and after a heap object. > Makes DoubleAlignForDeserialization explicitly fill after an already > aligned object. > > LOG=N > BUG=v8:4124 > > Committed: https://crrev.com/fcfb080eb9a637f0ae066bed4c45095e60df8a84 > Cr-Commit-Position: refs/heads/master@{#28687} TBR=hpayer@chromium.org,bmeurer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:4124 Review URL: https://codereview.chromium.org/1159123002 Cr-Commit-Position: refs/heads/master@{#28688}
-
bbudge authored
Moves alignment fill calculations into two static Heap methods. Adds a Heap method to handle the complex case where filler is potentially needed before and after a heap object. Makes DoubleAlignForDeserialization explicitly fill after an already aligned object. LOG=N BUG=v8:4124 Review URL: https://codereview.chromium.org/1150593003 Cr-Commit-Position: refs/heads/master@{#28687}
-
mbrandy authored
Port 3a1d7335 Original commit message: This finishes up work begun by CL (https://codereview.chromium.org/546683003/) some months ago. R=mvstanton@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1160933003 Cr-Commit-Position: refs/heads/master@{#28686}
-
mbrandy authored
R=bmeurer@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1143153013 Cr-Commit-Position: refs/heads/master@{#28685}
-
sergiyb authored
This will land with https://chromereviews.googleplex.com/202967013 R=akuegel@chromium.org, machenbach@chromium.org BUG=471767 NOTRY=true LOG=N Review URL: https://codereview.chromium.org/1161873004 Cr-Commit-Position: refs/heads/master@{#28684}
-
mvstanton authored
This finishes up work begun by CL (https://codereview.chromium.org/546683003/) some months ago. BUG= Review URL: https://codereview.chromium.org/1149053004 Cr-Commit-Position: refs/heads/master@{#28683}
-
bmeurer authored
This is an initial step towards a faster and less incorrect implementation of for-in in TurboFan. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1153403002 Cr-Commit-Position: refs/heads/master@{#28682}
-
mstarzinger authored
This introduces a conservative prediction for each exception handler whether it will locally catch an exception or re-throw it to outside the code bondaries. It will allow for a more intuitive prediction of whether an exception is considered "caught" or "uncaught". R=bmeurer@chromium.org,yangguo@chromium.org BUG=chromium:492522 LOG=N Review URL: https://codereview.chromium.org/1158563008 Cr-Commit-Position: refs/heads/master@{#28681}
-
erikcorry authored
R=hpayer@chromium.org BUG= Review URL: https://codereview.chromium.org/1158933007 Cr-Commit-Position: refs/heads/master@{#28680}
-
machenbach authored
NOTRY=true Review URL: https://codereview.chromium.org/1156663009 Cr-Commit-Position: refs/heads/master@{#28679}
-
jochen authored
BUG=4134 R=vogelheim@chromium.org LOG=n Review URL: https://codereview.chromium.org/1154423004 Cr-Commit-Position: refs/heads/master@{#28678}
-
bmeurer authored
We need the shared function info of inlined functions to prevent code flushing for their unoptimized code, and also to make sure that liveedit can find the proper functions to deoptimize. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1156403002 Cr-Commit-Position: refs/heads/master@{#28677}
-
machenbach authored
BUG=chromium:489254 LOG=n Review URL: https://codereview.chromium.org/1143173005 Cr-Commit-Position: refs/heads/master@{#28676}
-
bmeurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1156393002 Cr-Commit-Position: refs/heads/master@{#28675}
-
bmeurer authored
The control flow optimization should work independent of the JSGraph. We used the JSGraph there because it was convinient, not because it was necessary. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1160863003 Cr-Commit-Position: refs/heads/master@{#28674}
-
bmeurer authored
The SimplifiedOperatorReducer is (mostly) unused, except for the very rough store elimination, and just eats compilation time. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1162563002 Cr-Commit-Position: refs/heads/master@{#28673}
-
bmeurer authored
The list of inlined functions is used in exactly two places - for live edit and to prevent code flushing for inlined functions - and those are fine with SharedFunctionInfo and don't require a closure. This is one additional step towards inlining based on SharedFunctionInfo instead of JSFunction. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1145893003 Cr-Commit-Position: refs/heads/master@{#28672}
-
yangguo authored
R=arv@chromium.org BUG=chromium:492526 LOG=N Review URL: https://codereview.chromium.org/1146923004 Cr-Commit-Position: refs/heads/master@{#28671}
-
yangguo authored
R=mstarzinger@chromium.org BUG=chromium:490680 LOG=Y Review URL: https://codereview.chromium.org/1157563005 Cr-Commit-Position: refs/heads/master@{#28670}
-
bmeurer authored
Neither the increment nor the comparison can deoptimize, so we don't need proper frame states there. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1148323003 Cr-Commit-Position: refs/heads/master@{#28669}
-
v8-autoroll authored
Rolling v8/build/gyp to 29e94a3285ee899d14d5e56a6001682620d3778f Rolling v8/tools/clang to dc8f173e7fe10badbd9e29cd6eadc32ec552e691 TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1156373003 Cr-Commit-Position: refs/heads/master@{#28668}
-
- 27 May, 2015 17 commits
-
-
akos.palfi authored
This test is little-endian specific and therefore it can't run correctly on big-endian platforms. BUG= Review URL: https://codereview.chromium.org/1155113008 Cr-Commit-Position: refs/heads/master@{#28667}
-
mbrandy authored
Port 5450fc07 Original commit message: Also adapt code generation to pass the slot to the store/keyed-store ic. AST nodes ObjectLiteral, Assignment, ForEach, Call and CountOperation now include one or more feedback vector ic slot ids. R=mvstanton@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1156503003 Cr-Commit-Position: refs/heads/master@{#28666}
-
mbrandy authored
Port 44e98103 Original commit message: When we enter a method that needs access to the [[HomeObject]] we allocate a local variable `.home_object` and assign it the value from the [[HomeObject]] private symbol. Something along the lines of: method() { var .home_object = %ThisFunction()[home_object_symbol]; ... } R=arv@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1161603004 Cr-Commit-Position: refs/heads/master@{#28665}
-
conradw authored
Implements the strong mode proposal's restrictions on the ability of user code to modify the prototype of strong objects. Setting the strong bit is still wip, so this change will only affect those objects that have the bit correctly set. The tests reflect this, and will be expanded as more objects can be marked as strong. BUG=v8:3956 LOG=N Review URL: https://codereview.chromium.org/1143623002 Cr-Commit-Position: refs/heads/master@{#28664}
-
machenbach authored
BUG=chromium:489254 LOG=n Review URL: https://codereview.chromium.org/1158903002 Cr-Commit-Position: refs/heads/master@{#28663}
-
ulan authored
Make sure that when manual evacuation candidate selection is enabled we do not select more pages than requested. BUG= TBR=hpayer@chromium.org Review URL: https://codereview.chromium.org/1157293002 Cr-Commit-Position: refs/heads/master@{#28662}
-
vogelheim authored
The DCHECK was a lie. The idea was that - when a bookmark is set - the scanner must clearly be at a character boundary and hence the bookmark does not need to save a 'partial' UTF-8 code point. The first part is true - the Scanner is always at a character boundary - but the 'partial' UTF-8 code point is at the end of a block, not at the current character position of the Scanner. Hence, the 'partial' character needs to be saved as well. jkummerow: Thanks for noticing. BUG=chromium:470930 R=jochen@chromium.org, jkummerow@chromium.org LOG=N Review URL: https://codereview.chromium.org/1154773004 Cr-Commit-Position: refs/heads/master@{#28661}
-
jochen authored
Also deprecate ForceSet BUG=chromium:475206 R=adamk@chromium.org,verwaest@chromium.org LOG=y Review URL: https://codereview.chromium.org/1154233003 Cr-Commit-Position: refs/heads/master@{#28660}
-
mvstanton authored
Also adapt code generation to pass the slot to the store/keyed-store ic. AST nodes ObjectLiteral, Assignment, ForEach, Call and CountOperation now include one or more feedback vector ic slot ids. BUG= Review URL: https://codereview.chromium.org/1161623002 Cr-Commit-Position: refs/heads/master@{#28659}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1152153004 Cr-Commit-Position: refs/heads/master@{#28658}
-
v8-autoroll authored
Rolling v8/buildtools to a85661f97e83c620a8a1d0255829eb7a674e12cc TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1145043004 Cr-Commit-Position: refs/heads/master@{#28657}
-
machenbach authored
Revert of Use CLOCK_REALTIME_COARSE when available. (patchset #1 id:1 of https://codereview.chromium.org/1151283005/) Reason for revert: [Sheriff] This leads to several failures in chromium and blocks our roll: https://codereview.chromium.org/1154363002/ Bisect (https://codereview.chromium.org/1152553004/) points to this CL. Please add the failing chromium trybot on a reland of this CL. Original issue's description: > Use CLOCK_REALTIME_COARSE when available. > > On systems that have CLOCK_REALTIME_COARSE with good enough resolution, > we can avoid making a system call to get the current time; it's serviced > from the vDSO. > > This is v2 of the patch. v1 can be found at [0] but was reverted in [1] > because of Chromium sandbox restrictions. The necessary changes have > been applied upstream in [2]. > > [0] https://codereview.chromium.org/1125003002 > [1] https://codereview.chromium.org/1130083003 > [2] https://codereview.chromium.org/1133653002 > > BUG= > LOG=N > > Committed: https://crrev.com/28cea2b749f24ba33e6e0c8e343dd0d6258ee302 > Cr-Commit-Position: refs/heads/master@{#28639} TBR=jochen@chromium.org,bmeurer@chromium.org,ben@strongloop.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1154783003 Cr-Commit-Position: refs/heads/master@{#28656}
-
conradw authored
Strong Object/Array literals are currently being created with incorrect internal prototypes. This CL fixes this and extends the test suite to check. BUG= LOG=N Review URL: https://codereview.chromium.org/1158933002 Cr-Commit-Position: refs/heads/master@{#28655}
-
machenbach authored
BUG= Review URL: https://codereview.chromium.org/1153103003 Cr-Commit-Position: refs/heads/master@{#28654}
-
vegorov authored
R=jkummerow@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1157683007 Cr-Commit-Position: refs/heads/master@{#28653}
-
hpayer authored
Before we used to scale the growing factor based on freed global handles (which may have caused jank when many global handles got freed on site navigation). BUG= Review URL: https://codereview.chromium.org/1158433003 Cr-Commit-Position: refs/heads/master@{#28652}
-
ulan authored
This lifts the sqrt(n) limit on number of evacuation candidates, replaces O(n * sqrt(n)) algorithm with O(n*log(n)) algorithm, and removes hard-coded constants. Evacuation candidates are selected as follows: 1) Sort pages from the most free to the least free. 2) Select the first m pages as evacuation candidates such that m is as large as possible under the two conditions: - The total size of live objects in the first m pages does not exceed the given limit. This is based on the assumption that the evacuation cost is proportional to the total size of moved objects. - The fragmentation of the (m+1)-th page does not exceed the given limit. Review URL: https://codereview.chromium.org/1038313003 Cr-Commit-Position: refs/heads/master@{#28651}
-