Some cleanup of the toplevel code generator:...

src/x64/codegen-x64.cc

Index: src/x64/codegen-x64.cc
===================================================================
--- src/x64/codegen-x64.cc	(revision 3576)
+++ src/x64/codegen-x64.cc	(working copy)
@@ -4247,15 +4247,7 @@
     // The expression is either a property or a variable proxy that rewrites
     // to a property.
     Load(property->obj());
-    // We use a named reference if the key is a literal symbol, unless it is
-    // a string that can be legally parsed as an integer.  This is because
-    // otherwise we will not get into the slow case code that handles [] on
-    // String objects.
-    Literal* literal = property->key()->AsLiteral();
-    uint32_t dummy;
-    if (literal != NULL &&
-        literal->handle()->IsSymbol() &&
-        !String::cast(*(literal->handle()))->AsArrayIndex(&dummy)) {
+    if (property->key()->IsPropertyName()) {

[Lasse Reichstein, 2010/01/12 08:30:05]

This is a really great clean-up! So much easier to read and write.

       ref->set_type(Reference::NAMED);
     } else {
       Load(property->key());