[field type tracking] Fix handling of cleared WeakCells.

isheludko · Commit bot · commit 5ca478a5563b · 2015-12-23T17:09:38.000Z
Previous CL (https://codereview.chromium.org/1522413002) has a typo. BUG=chromium:571402,chromium:514080,chromium:527994,v8:4325 LOG=N Review URL: https://codereview.chromium.org/1541403002 Cr-Commit-Position: refs/heads/master@{#33031}
diff --git a/src/objects.cc b/src/objects.cc
@@ -3833,8 +3833,8 @@ MaybeHandle<Map> Map::TryUpdate(Handle<Map> old_map) {
       case DATA: {
         HeapType* new_type = new_descriptors->GetFieldType(i);
         // Cleared field types need special treatment. They represent lost
-        // knowledge, so we must first generalize the old_type to "Any".
-        if (!FieldTypeIsCleared(new_details.representation(), new_type)) {
+        // knowledge, so we must first generalize the new_type to "Any".
+        if (FieldTypeIsCleared(new_details.representation(), new_type)) {
           return MaybeHandle<Map>();
         }
         PropertyType old_property_type = old_details.type();
