Deoptimization support for accessors.

src/objects.cc

Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 569b5d030bdf4b3e16f697874db164a46110dc67..8cfb5fd1ffe0f6ff7bd3348e7ef09ac86819e1ac 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -8303,11 +8303,13 @@ void DeoptimizationInputData::DeoptimizationInputDataPrint(FILE* out) {
         }
 
         case Translation::ARGUMENTS_ADAPTOR_FRAME:
-        case Translation::CONSTRUCT_STUB_FRAME: {
+        case Translation::CONSTRUCT_STUB_FRAME:
+        case Translation::SETTER_STUB_FRAME: {

[Michael Starzinger, 2012/08/16 17:20:04]

Can we make that an own case that doesn't print the height at all? This makes it
hard to see when we expect the iterator to give us a height and when not.

[Sven Panne, 2012/08/17 07:27:26]

Done.

           int function_id = iterator.Next();
           JSFunction* function =
               JSFunction::cast(LiteralArray()->get(function_id));
-          unsigned height = iterator.Next();
+          unsigned height =
+              opcode == Translation::SETTER_STUB_FRAME ? 2 : iterator.Next();

[Michael Starzinger, 2012/08/16 17:20:04]

Actually, height = 0 for the setter stub, but that doesn't matter if the above
comment is addressed.

[Sven Panne, 2012/08/17 07:27:26]

Well, it is totally unclear what is *really* meant here: There is a lot of
naming confusion going on, this "height" here is sometimes called
"translation_size", at other places "height" has a different meaning, compare
e.g. LCodeGen::WriteTranslation and Translation::BeginConstructStubFrame. But as
you said, this won't matter here. :-)

           PrintF(out, "{function=");
           function->PrintName(out);
           PrintF(out, ", height=%u}", height);