Fix arguments object materialization during deopt.

src/arm/lithium-codegen-arm.h

Index: src/arm/lithium-codegen-arm.h
diff --git a/src/arm/lithium-codegen-arm.h b/src/arm/lithium-codegen-arm.h
index fd4a2a5ca7e07f80e8b756c27ce2d04059b3bedf..04669b04943c6cd5c2c8791e9d9df5ebede018b2 100644
--- a/src/arm/lithium-codegen-arm.h
+++ b/src/arm/lithium-codegen-arm.h
@@ -147,7 +147,10 @@ class LCodeGen BASE_EMBEDDED {
                                  int additional_offset);
 
   // Emit frame translation commands for an environment.
-  void WriteTranslation(LEnvironment* environment, Translation* translation);
+  void WriteTranslation(LEnvironment* environment,
+                        Translation* translation,
+                        int* arguments_index,

[Sven Panne, 2012/09/12 07:14:22]

I think later it will be very hard to understand what arguments_index actually
is, so we should document this somehow, but I am not sure where.

Perhaps arguments_offset might be a better name? I'll leave this up to you, but
we need some kind of comment in any case.

[Michael Starzinger, 2012/09/12 10:07:28]

Done. I added the documentation into LCodeGen::WriteTranslation where it's
actually used. About the name, I would still like to keep it named index,
because it is an operand index (same as LOperand::index) and follows the same
rules. That's in sync with the usage of "index" throughout lithium.

+                        int* arguments_count);
 
   // Declare methods that deal with the individual node types.
 #define DECLARE_DO(type) void Do##type(L##type* node);
@@ -258,7 +261,9 @@ class LCodeGen BASE_EMBEDDED {
   void AddToTranslation(Translation* translation,
                         LOperand* op,
                         bool is_tagged,
-                        bool is_uint32);
+                        bool is_uint32,
+                        int arguments_index,
+                        int arguments_count);
   void PopulateDeoptimizationData(Handle<Code> code);
   int DefineDeoptimizationLiteral(Handle<Object> literal);