Use ZoneHandle when creating ConstantInstr

runtime/vm/flow_graph_inliner.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/flow_graph_inliner.h"
 
 #include "vm/aot_optimizer.h"
 #include "vm/block_scheduler.h"
 #include "vm/branch_optimizer.h"
 #include "vm/compiler.h"
@@ skipping 3815 matching lines @@
             flow_graph->isolate()->class_table()->At(receiver_cid));
         if (!cls.IsGeneric()) {
           type = cls.CanonicalType();
         }
       }
 
       if (!type.IsNull()) {
           *entry = new(Z) TargetEntryInstr(flow_graph->allocate_block_id(),
                                            call->GetBlock()->try_index());
           (*entry)->InheritDeoptTarget(Z, call);
-          *last = new(Z) ConstantInstr(type);
+          *last = new(Z) ConstantInstr(Type::ZoneHandle(Z, type.raw()));

[siva, 2016/10/25 10:42:32]

The type handle seems to be created above, would it make sense to create it as a
ZoneHandle in the first place over there?

           flow_graph->AppendTo(*entry, *last,
                                call->deopt_id() != Thread::kNoDeoptId ?
                                call->env() : NULL,
                                FlowGraph::kValue);
           return true;
       }
       return false;
     }
 
     case MethodRecognizer::kOneByteStringSetAt: {
@@ skipping 22 matching lines @@
       return true;
     }
 
     default:
       return false;
   }
 }
 
 
 }  // namespace dart