Clone both descriptor and data array when compiling in background

runtime/vm/object.h

Index: runtime/vm/object.h
diff --git a/runtime/vm/object.h b/runtime/vm/object.h
index 9e6ba4209546f10f014100f3ed85cb96946e7e24..eb9cbb0a02ce5878a0e98e42930b98b3f7eb7f48 100644
--- a/runtime/vm/object.h
+++ b/runtime/vm/object.h
@@ -1945,7 +1945,7 @@ class ICData : public Object {
   // Entry array of the result is the same as in 'from'. Once entry array is
   // created, it can only change the 'count', all other properties are invariant
   // (target, cids, number of checks).
-  static RawICData* CloneDescriptor(const ICData& from);
+  static RawICData* CloneDescriptorAndData(const ICData& from);

[siva, 2016/02/12 22:10:25]

Why not call it Clone?

[srdjan, 2016/02/12 22:20:38]

Renamed to Clone

 
   static intptr_t TestEntryLengthFor(intptr_t num_args);
 
@@ -2681,9 +2681,9 @@ class Function : public Object {
       const ZoneGrowableArray<const ICData*>& deopt_id_to_ic_data,
       const Array& edge_counters_array) const;
   // Uses 'ic_data_array' to populate the table 'deopt_id_to_ic_data'. Clone
-  // descriptors if 'clone_descriptors' true.
+  // ic_data (array and descriptor) if 'clone_ic_data' is true.
   void RestoreICDataMap(ZoneGrowableArray<const ICData*>* deopt_id_to_ic_data,
-                        bool clone_descriptors) const;
+                        bool clone_ic_data) const;
 
   RawArray* ic_data_array() const;
   void ClearICDataArray() const;