Ensure we can reliably check the cell for validity of global property

src/ic/ia32/handler-compiler-ia32.cc

Index: src/ic/ia32/handler-compiler-ia32.cc
diff --git a/src/ic/ia32/handler-compiler-ia32.cc b/src/ic/ia32/handler-compiler-ia32.cc
index c02d83ce46fde9e3e01f2bef32c27996aa917013..0ae5706bdfd141b41c48b68706b25d123ffcef06 100644
--- a/src/ic/ia32/handler-compiler-ia32.cc
+++ b/src/ic/ia32/handler-compiler-ia32.cc
@@ -499,7 +499,8 @@ Register PropertyHandlerCompiler::CheckPrototypes(
   // Log the check depth.
   LOG(isolate(), IntEvent("check-maps-depth", depth + 1));
 
-  if (depth != 0 || check == CHECK_ALL_MAPS) {
+  if (!current_map->IsJSGlobalObjectMap() &&
+      (depth != 0 || check == CHECK_ALL_MAPS)) {
     // Check the holder map.
     __ mov(scratch1, FieldOperand(reg, HeapObject::kMapOffset));
     Handle<WeakCell> cell = Map::WeakCellForMap(current_map);
@@ -733,7 +734,7 @@ Register NamedStoreHandlerCompiler::value() {
 
 
 Handle<Code> NamedLoadHandlerCompiler::CompileLoadGlobal(
-    Handle<PropertyCell> cell, Handle<Name> name, bool is_configurable) {
+    Handle<PropertyCell> cell, Handle<Name> name) {
   Label miss;
   if (IC::ICUseVector(kind())) {
     PushVectorAndSlot();
@@ -745,14 +746,8 @@ Handle<Code> NamedLoadHandlerCompiler::CompileLoadGlobal(
   __ LoadWeakValue(result, weak_cell, &miss);
   __ mov(result, FieldOperand(result, PropertyCell::kValueOffset));
 
-  // Check for deleted property if property can actually be deleted.
-  if (is_configurable) {
-    __ cmp(result, factory()->the_hole_value());
-    __ j(equal, &miss);
-  } else if (FLAG_debug_code) {
-    __ cmp(result, factory()->the_hole_value());
-    __ Check(not_equal, kDontDeleteCellsCannotContainTheHole);
-  }
+  __ cmp(result, factory()->the_hole_value());
+  __ j(equal, &miss);
 
   Counters* counters = isolate()->counters();
   __ IncrementCounter(counters->named_load_global_stub(), 1);