Fix reintroduction of global variables that have been deleted.

src/arm/lithium-arm.cc

Index: src/arm/lithium-arm.cc
diff --git a/src/arm/lithium-arm.cc b/src/arm/lithium-arm.cc
index ce3a66f9c3fce0953f6d80a701087e49cb3aa2ec..5045ab8763eb3e2a521a0fbeff7d97a4283b9631 100644
--- a/src/arm/lithium-arm.cc
+++ b/src/arm/lithium-arm.cc
@@ -1602,7 +1602,12 @@ LInstruction* LChunkBuilder::DoLoadGlobal(HLoadGlobal* instr) {
 
 
 LInstruction* LChunkBuilder::DoStoreGlobal(HStoreGlobal* instr) {
-  return new LStoreGlobal(UseRegisterAtStart(instr->value()));
+  LOperand* temp = instr->check_hole_value() ? TempRegister() : NULL;

[Kevin Millikin (Chromium), 2011/01/27 07:56:07]

OK, but since almost every line has to check the same flag, I'd split it into:

if (instr->check_hole_value()) {
  LOperand* temp = TempRegister();
  LOperand* value = UseRegister(instr->value());
  LStoreGlobal* result = new LStoreGlobal(value, temp);
  return AssignEnvironment(result);
} else {
  LOperand* value = UseRegisterAtStart(instr->value());
  return new LStoreGlobal(value, NULL);
}

[Mads Ager (chromium), 2011/01/27 08:33:36]

Done.

+  LOperand* value = instr->check_hole_value()
+      ? UseRegister(instr->value())
+      : UseRegisterAtStart(instr->value());
+  LStoreGlobal* result = new LStoreGlobal(value, temp);
+  return instr->check_hole_value() ? AssignEnvironment(result) : result;
 }