danno
a988d5f261
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}
2015-05-05 16:42:16 +00:00
wingo
18619d3551
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
Review URL: https://codereview.chromium.org/1097283003
Cr-Commit-Position: refs/heads/master@{#28236}
2015-05-05 16:38:22 +00:00
machenbach
f6ca539952
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}
2015-05-05 15:45:07 +00:00
jochen
7e67b48694
freed_nodes in global-handles should be addititive
...
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}
2015-05-05 15:37:40 +00:00
erikcorry
49fdc53234
Be sure to abort incremental GC when trying to reserve space
...
R=hpayer@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1124553007
Cr-Commit-Position: refs/heads/master@{#28233}
2015-05-05 14:57:30 +00:00
mvstanton
9d0d7ec730
GrowArrayElementsStub must save caller doubles for double ElementsKind.
...
BUG=
R=danno@chromium.org
Review URL: https://codereview.chromium.org/1124653004
Cr-Commit-Position: refs/heads/master@{#28232}
2015-05-05 14:44:08 +00:00
jkummerow
0897244dd4
[grokdump] Update v8heapconst.py
...
Review URL: https://codereview.chromium.org/1125543004
Cr-Commit-Position: refs/heads/master@{#28231}
2015-05-05 14:26:35 +00:00
jkummerow
04cc497aba
[d8] Make Realm.dispose() trigger a ContextDisposedNotification
...
Review URL: https://codereview.chromium.org/1129543002
Cr-Commit-Position: refs/heads/master@{#28230}
2015-05-05 14:25:25 +00:00
kozyatinskiy
19b62e211d
[V8] Reland https://codereview.chromium.org/1100993003/
...
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}
2015-05-05 14:21:12 +00:00
jkummerow
367d14d467
[tick processor] Introduce --pairwise-timed-range processing mode
...
Review URL: https://codereview.chromium.org/1123883002
Cr-Commit-Position: refs/heads/master@{#28228}
2015-05-05 14:17:48 +00:00
machenbach
e47c362642
[test] Add arm perf trybot to runner script.
...
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}
2015-05-05 14:17:47 +00:00
arv
d26f5d3923
[es6] When comparing two symbols we may need to throw a TypeError
...
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}
2015-05-05 14:17:46 +00:00
ulan
f6d632bd4d
Reduce the size of initial JSON string buffer.
...
BUG=v8:3925
LOG=NO
Review URL: https://codereview.chromium.org/1123013002
Cr-Commit-Position: refs/heads/master@{#28225}
2015-05-05 14:17:45 +00:00
yangguo
a147377eaa
Do not cache object literal maps during bootstrapping.
...
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}
2015-05-05 14:17:44 +00:00
yangguo
7546302b67
Move more parts of stack trace formatting to runtime.
...
R=ishell@chromium.org
Review URL: https://codereview.chromium.org/1122973002
Cr-Commit-Position: refs/heads/master@{#28223}
2015-05-05 14:17:43 +00:00
bmeurer
3283195d04
[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
Review URL: https://codereview.chromium.org/1128533002
Cr-Commit-Position: refs/heads/master@{#28222}
2015-05-05 12:48:17 +00:00
ishell
1c74ca6619
When adding constant string with something unknown, assume it's a string.
...
Review URL: https://codereview.chromium.org/1122823002
Cr-Commit-Position: refs/heads/master@{#28221}
2015-05-05 12:47:08 +00:00
yangguo
728f0af66d
Fix bug in "Migrate error messages, part 7".
...
TBR=mvstanton@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1124583006
Cr-Commit-Position: refs/heads/master@{#28220}
2015-05-05 10:25:59 +00:00
bmeurer
d21de2a48b
[turbofan] Fix tail call optimization.
...
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}
2015-05-05 09:43:02 +00:00
hpayer
0090c05578
Add heap growing strategy details to --trace-gc-verbose.
...
BUG=
Review URL: https://codereview.chromium.org/1128483003
Cr-Commit-Position: refs/heads/master@{#28218}
2015-05-05 09:25:30 +00:00
yangguo
b0eb920fe2
Reland #2 "Wrap v8natives.js into a function."
...
TBR=machenbach@chromium.org
Review URL: https://codereview.chromium.org/1127693006
Cr-Commit-Position: refs/heads/master@{#28217}
2015-05-05 09:16:01 +00:00
yangguo
a3ddb1bc42
Migrate error messages, part 7.
...
R=mvstanton@chromium.org
Review URL: https://codereview.chromium.org/1122033002
Cr-Commit-Position: refs/heads/master@{#28216}
2015-05-05 07:57:36 +00:00
danno
f36ecaf3a4
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.
Review URL: https://codereview.chromium.org/1053143005
Cr-Commit-Position: refs/heads/master@{#28215}
2015-05-05 07:55:58 +00:00
bmeurer
64d7340439
[c++11] Remove remnants of pre-C++11 delete, final and override.
...
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}
2015-05-05 07:44:35 +00:00
yangguo
7f927ced51
Correctly accept already preprocessed stack traces.
...
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}
2015-05-05 07:28:54 +00:00
michael_dawson
785845e0a1
Fix another may not be init compile error on AIX
...
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}
2015-05-05 07:17:47 +00:00
alph
6964a9e068
Make CPU profiler do not hog 100% of CPU.
...
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}
2015-05-04 22:53:28 +00:00
dehrenberg
d5e66a70c3
comment typo fix
...
Review URL: https://codereview.chromium.org/1120373003
Cr-Commit-Position: refs/heads/master@{#28210}
2015-05-04 19:45:15 +00:00
mbrandy
f5fe8fd707
PPC: Fix 'Remove materialized objects on stack unwind.'
...
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}
2015-05-04 19:30:07 +00:00
machenbach
6afc0dcbfc
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}
2015-05-04 16:49:58 +00:00
machenbach
17dca16df3
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}
2015-05-04 16:47:54 +00:00
jarin
b5b47e1f88
Remove materialized objects on stack unwind.
...
BUG=v8:3985
LOG=n
R=mstarzinger@chromium.org
Review URL: https://codereview.chromium.org/1122083002
Cr-Commit-Position: refs/heads/master@{#28206}
2015-05-04 16:44:01 +00:00
machenbach
d0ed8940fc
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}
2015-05-04 15:39:46 +00:00
machenbach
0fa31f75cd
[test] Skip some flaky layout tests.
...
NOTRY=true
TBR=jochen@chromium.org
Review URL: https://codereview.chromium.org/1126433004
Cr-Commit-Position: refs/heads/master@{#28204}
2015-05-04 15:38:36 +00:00
mbrandy
b10fb64b0c
PPC: Only swap undefined for the global object if necessary in the prologue
...
Port fab3508062R=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}
2015-05-04 15:33:56 +00:00
verwaest
f21ea065b3
Fix smi scanning
...
BUG=chromium:483176
LOG=n
Review URL: https://codereview.chromium.org/1114073003
Cr-Commit-Position: refs/heads/master@{#28202}
2015-05-04 15:02:30 +00:00
akos.palfi
2cdd74b3ae
MIPS: Skip test-simplified-lowering/RunNumberDivide_2_TruncatingToUint32.
...
This test fails on board, temporarily skip failing test until we resolve this issue.
NOTRY=true
Review URL: https://codereview.chromium.org/1127573002
Cr-Commit-Position: refs/heads/master@{#28201}
2015-05-04 14:14:36 +00:00
verwaest
fab3508062
Only swap undefined for the global object if necessary in the prologue
...
BUG=
Review URL: https://codereview.chromium.org/1120093002
Cr-Commit-Position: refs/heads/master@{#28200}
2015-05-04 13:39:00 +00:00
hpayer
6e9e2c0829
New insertion write barrier.
...
BUG=
Review URL: https://codereview.chromium.org/1073953006
Cr-Commit-Position: refs/heads/master@{#28199}
2015-05-04 13:12:26 +00:00
mstarzinger
120344ec44
[turbofan] Free TurboFan from the claws of ZoneList.
...
This removes the last occurence of the hand-written ZoneList from the
compiler directory and uses ZoneVector everywhere instead.
R=bmeurer@chromium.org
Review URL: https://codereview.chromium.org/1120083004
Cr-Commit-Position: refs/heads/master@{#28198}
2015-05-04 12:54:15 +00:00
hablich
276a846f69
[release-tools] Tool to find related commits
...
usage: This tool analyzes the commit range between <of> and <until>. It finds commits which belong together e.g. Implement/Revert pairs and Implement/Port/Revert triples. All supplied hashes need to be from the same branch e.g. master.
Example for M42: ./search_related_commits.py --prettyprint --separator e0110920d6b856e87859b1c2a34956https://codereview.chromium.org/1098123002
Cr-Commit-Position: refs/heads/master@{#28197}
2015-05-04 12:20:16 +00:00
hpayer
fff4f8e1bf
Don't perform marking barrier for weak cell values.
...
BUG=
Review URL: https://codereview.chromium.org/1124583003
Cr-Commit-Position: refs/heads/master@{#28196}
2015-05-04 12:15:41 +00:00
mstarzinger
d6945db703
[turbofan] Add SimplifiedOperator::Allocate operator.
...
This introduces a simplified allocation operator which can be used to
model inline allocations in TurboFan. It is currently used for context
allocations, but still disabled because change lowering introduces
floating allocations outside the effect chain that interfere.
R=bmeurer@chromium.org
Review URL: https://codereview.chromium.org/1109773002
Cr-Commit-Position: refs/heads/master@{#28195}
2015-05-04 12:07:12 +00:00
akos.palfi
900bc7937c
MIPS: Do not use the 64-bit floor and truncate instructions in fp32 mode.
...
These instructions have unpredictable result when the processor is in fp32 mode.
BUG=
TEST=mjsunit/math-floor-global,math-floor-local,math-floor-part1
Review URL: https://codereview.chromium.org/1116073002
Cr-Commit-Position: refs/heads/master@{#28194}
2015-05-04 12:06:03 +00:00
yangguo
72ab421729
Reland "Wrap v8natives.js into a function."
...
Review URL: https://codereview.chromium.org/1123703002
Cr-Commit-Position: refs/heads/master@{#28193}
2015-05-04 11:17:01 +00:00
titzer
9833844268
[turbofan] Small fixes to PipelineStatistics. Collect statistics in tests.
...
R=bmeurer@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1122753002
Cr-Commit-Position: refs/heads/master@{#28192}
2015-05-04 11:05:45 +00:00
titzer
acdb5336a9
Extract Signature from src/compiler/machine-type.h to src/signature.h
...
R=bmeurer@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1118823003
Cr-Commit-Position: refs/heads/master@{#28191}
2015-05-04 10:11:54 +00:00
yangguo
f42544b768
Set inferred name of bound function to empty string.
...
Otherwise it's whatever the js minifier assigns it to.
R=jkummerow@chromium.org
BUG=chromium:484077
LOG=N
Review URL: https://codereview.chromium.org/1122733002
Cr-Commit-Position: refs/heads/master@{#28190}
2015-05-04 09:55:43 +00:00
chunyang.dai
d09e119bc4
X87: VectorICs: built-in function apply should use an IC.
...
port 83a0af5500https://codereview.chromium.org/1119263002
Cr-Commit-Position: refs/heads/master@{#28189}
2015-05-04 03:40:28 +00:00
chunyang.dai
906152b85a
X87: Use a stub in crankshaft for grow store arrays.
...
port fb8e613638https://codereview.chromium.org/368263003
BUG=
Review URL: https://codereview.chromium.org/1125623002
Cr-Commit-Position: refs/heads/master@{#28188}
2015-05-04 03:39:19 +00:00
