Revision: 20874
Author:   [email protected]
Date:     Tue Apr 22 08:34:44 2014 UTC
Log:      Make sure to clear any previously collected field maps.

TEST=http://www.chaostoperfection.com/
[email protected]

Review URL: https://codereview.chromium.org/246963003
http://code.google.com/p/v8/source/detail?r=20874

Modified:
 /branches/bleeding_edge/src/hydrogen.cc

=======================================
--- /branches/bleeding_edge/src/hydrogen.cc     Thu Apr 17 14:20:26 2014 UTC
+++ /branches/bleeding_edge/src/hydrogen.cc     Tue Apr 22 08:34:44 2014 UTC
@@ -5574,15 +5574,15 @@

 void HOptimizedGraphBuilder::PropertyAccessInfo::LoadFieldMaps(
     Handle<Map> map) {
+  // Clear any previously collected field maps.
+  field_maps_.Clear();
+
   // Figure out the field type from the accessor map.
Handle<HeapType> field_type(lookup_.GetFieldTypeFromMap(*map), isolate());

   // Collect the (stable) maps from the field type.
   int num_field_maps = field_type->NumClasses();
-  if (num_field_maps == 0) {
-    field_maps_.Clear();
-    return;
-  }
+  if (num_field_maps == 0) return;
   ASSERT(access_.representation().IsHeapObject());
   field_maps_.Reserve(num_field_maps, zone());
   HeapType::Iterator<Map> it = field_type->Classes();

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to