X64 Crankshaft: Implement LValueOf and (dummy) LArgumentsObject.

src/ia32/lithium-ia32.cc

Index: src/ia32/lithium-ia32.cc
===================================================================
--- src/ia32/lithium-ia32.cc	(revision 6847)
+++ src/ia32/lithium-ia32.cc	(working copy)
@@ -1974,8 +1974,10 @@
 
 
 LInstruction* LChunkBuilder::DoArgumentsObject(HArgumentsObject* instr) {
-  // There are no real uses of the arguments object (we bail out in all other
-  // cases).
+  // There are no real uses of the arguments object.
+  // arguments.length and element access are supported directly on
+  // stack arguments, and any real arguments object use causes a bailout.
+  // So this value is never used.
   return NULL;

[Lasse Reichstein, 2011/02/18 11:50:55]

What is the "value" that is never used?
Is this function never called, or is its return value never used (and if so, how
do we know that)?
If it's never called, it should either assert that, or do a bailout if it ever
got called anyway.
If the return value isn't used, why are we calling the function?

[William Hesse, 2011/02/18 12:06:21]

A HValue, HArgumentsObject is created to represent the arguments object.  Once
we figure it out, we will use this object.  Currently, we don't compile any uses
of it in Crankshaft, so the HValue turns into zero lithium instructions.  The
way to create zero lithium instructions for an HValue is to return null from
DoH...().
In the long run, this should be implemented or HArgumentsObject should be
removed.


On 2011/02/18 11:50:55, Lasse Reichstein wrote:

 }