Improve load of global variables.

src/ia32/codegen-ia32.cc

Index: src/ia32/codegen-ia32.cc
===================================================================
--- src/ia32/codegen-ia32.cc	(revision 3908)
+++ src/ia32/codegen-ia32.cc	(working copy)
@@ -689,6 +689,9 @@
     // The expression is a variable proxy that does not rewrite to a
     // property.  Global variables are treated as named property references.
     if (var->is_global()) {
+      // Named loads require object in eax.  Named stores don't use references.
+      // Spilling eax makes it free, so LoadGlobal loads directly into eax.

[Kevin Millikin (Chromium), 2010/02/19 10:23:04]

The comment is still not quite right.  LoadIC wants receiver in eax and the
register allocator will prefer eax if free.

I think it's obvious that Spill(eax) makes it free, but not that the register
allocator prefers it.

Named stores do use reference in the case of e.x++ and ++e.x, so don't put that
comment at all since it's not true.  

+      frame_->Spill(eax);
       LoadGlobal();
       ref->set_type(Reference::NAMED);
     } else {