Reviewers: titzer,
Message:
Could you take a look, please?
Description:
Relax representation requirement in FrameStates.
This change enables non-tagged representations in FrameStates.
That allows us to run zlib with deoptimization support and have almost the
same
performance of the generated code (as the code with no deoptimization).
Unfortunately, the frame states seem to confuse typer. As a consequence, we
generate more representation changes, which in turn causes the scheduler to
take
a lot more time and memory (>4x). The added compiler time makes zlib with
deopt
be about 50% slower.
BUG=
Please review this at https://codereview.chromium.org/614713002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+341, -93 lines):
M src/compiler/code-generator.h
M src/compiler/code-generator.cc
M src/compiler/instruction.h
M src/compiler/instruction.cc
M src/compiler/instruction-selector.h
M src/compiler/instruction-selector.cc
M src/compiler/machine-type.h
M src/compiler/simplified-lowering.cc
M test/unittests/compiler/instruction-selector-unittest.h
M test/unittests/compiler/instruction-selector-unittest.cc
--
--
v8-dev mailing list
[email protected]
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 [email protected].
For more options, visit https://groups.google.com/d/optout.
