Thanks for the first round of comments!
mstarzinger, could you please take a look at js-inlining.cc?
https://codereview.chromium.org/573703002/diff/20001/src/compiler/code-generator.cc
File src/compiler/code-generator.cc (right):
https://codereview.chromium.org/573703002/diff/20001/src/compiler/code-generator.cc#newcode330
src/compiler/code-generator.cc:330:
descriptor->outer_state()->GetTotalSize() + i));
On 2014/09/15 13:37:51, jarin wrote:
How about adding descriptor->outer_state()->GetTotalSize() to
frame_state_offset
outside the loop?
Done.
https://codereview.chromium.org/573703002/diff/20001/src/compiler/instruction-selector.cc
File src/compiler/instruction-selector.cc (right):
https://codereview.chromium.org/573703002/diff/20001/src/compiler/instruction-selector.cc#newcode1043
src/compiler/instruction-selector.cc:1043: if (descriptor->type() ==
JS_FRAME) {
On 2014/09/15 13:37:51, jarin wrote:
How about introducing FrameStateDescriptor::HasContext method?
Done.
https://codereview.chromium.org/573703002/diff/20001/src/compiler/node-properties-inl.h
File src/compiler/node-properties-inl.h (right):
https://codereview.chromium.org/573703002/diff/20001/src/compiler/node-properties-inl.h#newcode166
src/compiler/node-properties-inl.h:166: Node* frame_state) {
On 2014/09/15 13:37:51, jarin wrote:
Perhaps an assert here that the node indeed has frame state input?
Done.
https://codereview.chromium.org/573703002/diff/20001/src/frames.cc
File src/frames.cc (right):
https://codereview.chromium.org/573703002/diff/20001/src/frames.cc#newcode937
src/frames.cc:937: if (LookupCode()->is_turbofanned() &&
!FLAG_turbo_deoptimization) {
On 2014/09/15 13:37:51, jarin wrote:
This is a bold move. We should try to run with run-tests
--extra-flags="--turbo-deoptimization" and see what breaks with this.
Do not the GetInlineCount and GetFunctions methods below need similar
treatment?
Most of the errors I'm getting seem to be missing deopt points; we need
to fix this later.
https://codereview.chromium.org/573703002/diff/20001/test/cctest/compiler/test-run-inlining.cc
File test/cctest/compiler/test-run-inlining.cc (right):
https://codereview.chromium.org/573703002/diff/20001/test/cctest/compiler/test-run-inlining.cc#newcode95
test/cctest/compiler/test-run-inlining.cc:95: "= 12;"
On 2014/09/15 13:37:52, jarin wrote:
This is a funky line break. Could you make it a bit more readable?
Done.
https://codereview.chromium.org/573703002/
--
--
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.
