Reviewers: rossberg,
Message:
PTAL
Description:
[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
Please review this at https://codereview.chromium.org/1123043002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+393, -118 lines):
M src/arm/lithium-arm.h
M src/arm/lithium-codegen-arm.cc
M src/arm64/lithium-arm64.h
M src/arm64/lithium-codegen-arm64.cc
M src/hydrogen.cc
M src/hydrogen-instructions.h
M src/hydrogen-instructions.cc
M src/ia32/lithium-codegen-ia32.cc
M src/ia32/lithium-ia32.h
M src/mips/lithium-codegen-mips.cc
M src/mips/lithium-mips.h
M src/mips64/lithium-codegen-mips64.cc
M src/mips64/lithium-mips64.h
M src/ppc/lithium-codegen-ppc.cc
M src/ppc/lithium-ppc.h
M src/x64/lithium-codegen-x64.cc
M src/x64/lithium-x64.h
M src/x87/lithium-codegen-x87.cc
M src/x87/lithium-x87.h
M test/mjsunit/strong/implicit-conversions.js
A test/mjsunit/strong/implicit-conversions-inlining.js
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
