|
[strong] Fix inlining issue
The Hydrogen representation for binops was never changed to care about the
language mode. We thought this was ok, but it turns out we need to keep track
of it to make sure inlining doesn't mess with the "strongness" of binops.
Also added more rigorous inlining testing.
BUG= v8:3956
LOG=N
Committed: https://crrev.com/97bee8e9644151cf6560c79991b82862066e1eb7
Cr-Commit-Position: refs/heads/master@{#28253}
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+393 lines, -118 lines) |
Patch |
|
M |
src/arm/lithium-arm.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
src/arm/lithium-codegen-arm.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/arm64/lithium-arm64.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
src/arm64/lithium-codegen-arm64.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/hydrogen.cc
|
View
|
|
8 chunks |
+19 lines, -17 lines |
0 comments
|
Download
|
|
M |
src/hydrogen-instructions.h
|
View
|
|
29 chunks |
+78 lines, -46 lines |
0 comments
|
Download
|
|
M |
src/hydrogen-instructions.cc
|
View
|
|
11 chunks |
+21 lines, -13 lines |
0 comments
|
Download
|
|
M |
src/ia32/lithium-codegen-ia32.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/ia32/lithium-ia32.h
|
View
|
|
1 chunk |
+4 lines, -1 line |
0 comments
|
Download
|
|
M |
src/mips/lithium-codegen-mips.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/mips/lithium-mips.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
src/mips64/lithium-codegen-mips64.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/mips64/lithium-mips64.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
src/ppc/lithium-codegen-ppc.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/ppc/lithium-ppc.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
src/x64/lithium-codegen-x64.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/x64/lithium-x64.h
|
View
|
|
1 chunk |
+4 lines, -0 lines |
0 comments
|
Download
|
|
M |
src/x87/lithium-codegen-x87.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
src/x87/lithium-x87.h
|
View
|
|
1 chunk |
+4 lines, -1 line |
0 comments
|
Download
|
|
M |
test/mjsunit/strong/implicit-conversions.js
|
View
|
|
2 chunks |
+0 lines, -32 lines |
0 comments
|
Download
|
|
A |
test/mjsunit/strong/implicit-conversions-inlining.js
|
View
|
|
1 chunk |
+235 lines, -0 lines |
0 comments
|
Download
|
Total messages: 8 (3 generated)
|