- 05 May, 2015 31 commits
-
-
dslomov authored
ArgumentsAdaptorStub for derived constructor (the one that needs new.target) works in this way: - If the constructor is invoked via the Construct stub, we know that actual arguments always include new.target. ``arguments`` object however should not include a new.target, therefore we remove it. We achieve this by decrementing the argument count. - If the constructor is invoked as a call, we do not care for a correct ``arguments`` array since the constructor will immediately throw on entrance. The bug is that the call could actually pass 0 actual arguments, but I decrement unconditionally :(. The fix is to detect this case and avoid decrementing. ``arguments`` is bogus, but it is ok as constructor throws. Long-term we should just remove mucking about with arguments for new.target and just get it from the stack. R=arv@chromium.org,rossberg@chromium.org BUG=chromium:474783 LOG=Y Review URL: https://codereview.chromium.org/1126783003 Cr-Commit-Position: refs/heads/master@{#28242}
-
dusan.milosavljevic authored
- Add long branche mode for FPU branches. - Fix FPU branches for unordered conditions. - Provide FPU [un]ordered condition negation schema. TEST=mjsunit/miror-objects, constant-folding-2, external-array BUG= Review URL: https://codereview.chromium.org/1120753010 Cr-Commit-Position: refs/heads/master@{#28241}
-
balazs.kilvady authored
Improve d26f5d39 Original commit message: When comparing a symbol to istself using <, <=, > or >= we need to throw a TypeError. This is correctly handled in the runtime function so if we are comparing a symbol fall back to use the runtime. BUG= Review URL: https://codereview.chromium.org/1124863004 Cr-Commit-Position: refs/heads/master@{#28240}
-
mbrandy authored
Port d21de2a4 Original commit message: Tail calls are matched on the graph, with a dedicated tail call optimization that is actually testable. The instruction selection can still fall back to a regular if the platform constraints don't allow to emit a tail call (i.e. the return locations of caller and callee differ or the callee takes non-register parameters, which is a restriction that will be removed in the future). Also explicitly limit tail call optimization to stubs for now and drop the global flag. R=dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1114193003 Cr-Commit-Position: refs/heads/master@{#28239}
-
wingo authored
Revert of Resolve references to "this" the same way as normal variables (patchset #11 id:240001 of https://codereview.chromium.org/1097283003/) Reason for revert: nosnap failures Original issue's description: > Resolve references to "this" the same way as normal variables > > Make the parser handle references to "this" as unresolved variables, so the > same logic as for the rest of function parameters is used for the receiver. > Minor additions to the code generation handle copying the receiver to the > context, along with the rest of the function parameters. > > Based on work by Adrian Perez de Castro <aperez@igalia.com>. > > BUG= > LOG=N > > Committed: https://crrev.com/18619d355192e2699203d12d9ebb9caea107b693 > Cr-Commit-Position: refs/heads/master@{#28236} TBR=rossberg@chromium.org,mstarzinger@chromium.org,dslomov@chromium.org,adamk@chromium.org,arv@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1113133006 Cr-Commit-Position: refs/heads/master@{#28238}
-
danno authored
Revert of Collect type feedback on result of Math.[round|ceil|floor] (patchset #13 id:230001 of https://codereview.chromium.org/1053143005/) Reason for revert: All sorts of performance regressions Original issue's description: > Collect type feedback on result of Math.[round|ceil|floor] > > By recording invocations of these builtins that can return -0, we now learn to not emit Crankshaft code that only handles integer results, avoiding deopt loops. > > Committed: https://crrev.com/f36ecaf3a4d61568ca50a20718acce7dd5da9a5f > Cr-Commit-Position: refs/heads/master@{#28215} TBR=mvstanton@chromium.org,bmeurer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1115973005 Cr-Commit-Position: refs/heads/master@{#28237}
-
wingo authored
Make the parser handle references to "this" as unresolved variables, so the same logic as for the rest of function parameters is used for the receiver. Minor additions to the code generation handle copying the receiver to the context, along with the rest of the function parameters. Based on work by Adrian Perez de Castro <aperez@igalia.com>. BUG= LOG=N Review URL: https://codereview.chromium.org/1097283003 Cr-Commit-Position: refs/heads/master@{#28236}
-
machenbach authored
Revert of [handles] Sanitize Handle and friends. (patchset #4 id:50001 of https://codereview.chromium.org/1128533002/) Reason for revert: [Sheriff] Breaks mac asan: http://build.chromium.org/p/client.v8/builders/V8%20Mac64%20ASAN/builds/1198 Original issue's description: > [handles] Sanitize Handle and friends. > > This almost allows us to get rid of handles-inl.h, once we fix the > cyclic dependencies (follow up CL). > > R=yangguo@chromium.org > > Committed: https://crrev.com/3283195d0408333cce552cf4087577e6f41054e5 > Cr-Commit-Position: refs/heads/master@{#28222} TBR=yangguo@chromium.org,bmeurer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1128543004 Cr-Commit-Position: refs/heads/master@{#28235}
-
jochen authored
otherwise phantom nodes won't ever be taken into account BUG=chromium:484671,chromium:479796 R=hpayer@chromium.org LOG=y Review URL: https://codereview.chromium.org/1125673010 Cr-Commit-Position: refs/heads/master@{#28234}
-
erikcorry authored
R=hpayer@chromium.org BUG= Review URL: https://codereview.chromium.org/1124553007 Cr-Commit-Position: refs/heads/master@{#28233}
-
mvstanton authored
BUG= R=danno@chromium.org Review URL: https://codereview.chromium.org/1124653004 Cr-Commit-Position: refs/heads/master@{#28232}
-
jkummerow authored
Review URL: https://codereview.chromium.org/1125543004 Cr-Commit-Position: refs/heads/master@{#28231}
-
jkummerow authored
Review URL: https://codereview.chromium.org/1129543002 Cr-Commit-Position: refs/heads/master@{#28230}
-
https://codereview.chromium.org/1100993003/kozyatinskiy authored
Chromium tests were prepared in https://codereview.chromium.org/1118743003/ Blink tests in https://codereview.chromium.org/1115193002/ LOG=Y BUG=chromium:480652 R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1121833003 Cr-Commit-Position: refs/heads/master@{#28229}
-
jkummerow authored
Review URL: https://codereview.chromium.org/1123883002 Cr-Commit-Position: refs/heads/master@{#28228}
-
machenbach authored
BUG=chromium:484589 LOG=n NOTRY=true TBR=hpayer@chromium.org Review URL: https://codereview.chromium.org/1123473007 Cr-Commit-Position: refs/heads/master@{#28227}
-
arv authored
When comparing a symbol to istself using <, <=, > or >= we need to throw a TypeError. This is correctly handled in the runtime function so if we are comparing a symbol fall back to use the runtime. BUG=v8:4073 LOG=Y R=rossberg@chromium.org Review URL: https://codereview.chromium.org/1125783002 Cr-Commit-Position: refs/heads/master@{#28226}
-
ulan authored
BUG=v8:3925 LOG=NO Review URL: https://codereview.chromium.org/1123013002 Cr-Commit-Position: refs/heads/master@{#28225}
-
yangguo authored
We do not expect to share maps for object literals created by builtin Javascript code. R=ishell@chromium.org Review URL: https://codereview.chromium.org/1127693009 Cr-Commit-Position: refs/heads/master@{#28224}
-
yangguo authored
R=ishell@chromium.org Review URL: https://codereview.chromium.org/1122973002 Cr-Commit-Position: refs/heads/master@{#28223}
-
bmeurer authored
This almost allows us to get rid of handles-inl.h, once we fix the cyclic dependencies (follow up CL). R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1128533002 Cr-Commit-Position: refs/heads/master@{#28222}
-
ishell authored
Review URL: https://codereview.chromium.org/1122823002 Cr-Commit-Position: refs/heads/master@{#28221}
-
yangguo authored
TBR=mvstanton@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1124583006 Cr-Commit-Position: refs/heads/master@{#28220}
-
bmeurer authored
Tail calls are matched on the graph, with a dedicated tail call optimization that is actually testable. The instruction selection can still fall back to a regular if the platform constraints don't allow to emit a tail call (i.e. the return locations of caller and callee differ or the callee takes non-register parameters, which is a restriction that will be removed in the future). Also explicitly limit tail call optimization to stubs for now and drop the global flag. BUG=v8:4076 LOG=n Review URL: https://codereview.chromium.org/1114163005 Cr-Commit-Position: refs/heads/master@{#28219}
-
hpayer authored
BUG= Review URL: https://codereview.chromium.org/1128483003 Cr-Commit-Position: refs/heads/master@{#28218}
-
yangguo authored
TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1127693006 Cr-Commit-Position: refs/heads/master@{#28217}
-
yangguo authored
R=mvstanton@chromium.org Review URL: https://codereview.chromium.org/1122033002 Cr-Commit-Position: refs/heads/master@{#28216}
-
danno authored
By recording invocations of these builtins that can return -0, we now learn to not emit Crankshaft code that only handles integer results, avoiding deopt loops. Review URL: https://codereview.chromium.org/1053143005 Cr-Commit-Position: refs/heads/master@{#28215}
-
bmeurer authored
These macros are not needed anymore, so there's no point in supporting them. Review URL: https://codereview.chromium.org/1123723003 Cr-Commit-Position: refs/heads/master@{#28214}
-
yangguo authored
When we preprocess stack traces, we turn code pointer and offset to source position, and store it in place of code pointer as smi. Preprocessing stack traces is currently disabled due to issue 4065. R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1125723002 Cr-Commit-Position: refs/heads/master@{#28213}
-
michael_dawson authored
AIX compiler is complaining that variable may not be initialized modified: src/heap/heap.cc R=danno@chromium.org, svenpanne@chromium.org, mbrandy@us.ibm.com BUG= Review URL: https://codereview.chromium.org/1123743002 Cr-Commit-Position: refs/heads/master@{#28212}
-
- 04 May, 2015 9 commits
-
-
alph authored
Tick event processor should not stay in a tight loop when there's nothing to do. It can go sleep until next sample event. LOG=N BUG=v8:3967 Review URL: https://codereview.chromium.org/1118533003 Cr-Commit-Position: refs/heads/master@{#28211}
-
dehrenberg authored
Review URL: https://codereview.chromium.org/1120373003 Cr-Commit-Position: refs/heads/master@{#28210}
-
mbrandy authored
R=mstarzinger@chromium.org, jarin@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1123793002 Cr-Commit-Position: refs/heads/master@{#28209}
-
machenbach authored
Revert of Reland "Wrap v8natives.js into a function." (patchset #2 id:20001 of https://codereview.chromium.org/1123703002/) Reason for revert: [Sheriff] Speculative revert for braking arm64 nosnap: http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20nosnap%20-%20debug%20-%202/builds/2314 (reverted already titzer's CL which didn't help) Original issue's description: > Reland "Wrap v8natives.js into a function." > > Committed: https://crrev.com/72ab42172979b60a1b784ea0c6a495d7ee2bba67 > Cr-Commit-Position: refs/heads/master@{#28193} TBR=jkummerow@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1127543003 Cr-Commit-Position: refs/heads/master@{#28208}
-
machenbach authored
Revert of Revert of [turbofan] Small fixes to PipelineStatistics. Collect statistics in tests. (patchset #1 id:1 of https://codereview.chromium.org/1119963003/) Reason for revert: Revert didn't fix the problem. Maybe it was another CL. Original issue's description: > Revert of [turbofan] Small fixes to PipelineStatistics. Collect statistics in tests. (patchset #1 id:1 of https://codereview.chromium.org/1122753002/) > > Reason for revert: > [Sheriff] Speculative revert for breaking arm64 nosnap: > http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20nosnap%20-%20debug%20-%202/builds/2314 > > Original issue's description: > > [turbofan] Small fixes to PipelineStatistics. Collect statistics in tests. > > > > R=bmeurer@chromium.org > > BUG= > > > > Committed: https://crrev.com/9833844268357590243732a53e2a7fc0dc5fcc9e > > Cr-Commit-Position: refs/heads/master@{#28192} > > TBR=bmeurer@chromium.org,titzer@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG= > > Committed: https://crrev.com/d0ed8940fc6c995eba57149b8e8cb214001e8381 > Cr-Commit-Position: refs/heads/master@{#28205} TBR=bmeurer@chromium.org,titzer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1126603002 Cr-Commit-Position: refs/heads/master@{#28207}
-
jarin authored
BUG=v8:3985 LOG=n R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1122083002 Cr-Commit-Position: refs/heads/master@{#28206}
-
machenbach authored
Revert of [turbofan] Small fixes to PipelineStatistics. Collect statistics in tests. (patchset #1 id:1 of https://codereview.chromium.org/1122753002/) Reason for revert: [Sheriff] Speculative revert for breaking arm64 nosnap: http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20nosnap%20-%20debug%20-%202/builds/2314 Original issue's description: > [turbofan] Small fixes to PipelineStatistics. Collect statistics in tests. > > R=bmeurer@chromium.org > BUG= > > Committed: https://crrev.com/9833844268357590243732a53e2a7fc0dc5fcc9e > Cr-Commit-Position: refs/heads/master@{#28192} TBR=bmeurer@chromium.org,titzer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1119963003 Cr-Commit-Position: refs/heads/master@{#28205}
-
machenbach authored
NOTRY=true TBR=jochen@chromium.org Review URL: https://codereview.chromium.org/1126433004 Cr-Commit-Position: refs/heads/master@{#28204}
-
mbrandy authored
Port fab35080 R=verwaest@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1122773002 Cr-Commit-Position: refs/heads/master@{#28203}
-