Address review comments of previously submitted cl 8329005.

runtime/vm/code_generator.cc

 // Copyright (c) 2011, 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/code_generator.h"
 
 #include "vm/code_index_table.h"
 #include "vm/code_patcher.h"
 #include "vm/compiler.h"
 #include "vm/dart_entry.h"
@@ skipping 131 matching lines @@
 // Allocate a new object.
 // Arg0: class of the object that needs to be allocated.
 // Arg1: type arguments of the object that needs to be allocated.
 // Arg2: type arguments of the instantiator.
 // Return value: newly allocated object.
 DEFINE_RUNTIME_ENTRY(AllocateObject, 3) {
   ASSERT(arguments.Count() == kAllocateObjectRuntimeEntry.argument_count());
   const Class& cls = Class::CheckedHandle(arguments.At(0));
   const Instance& instance = Instance::Handle(Instance::New(cls));
   arguments.SetReturn(instance);
-  const intptr_t num_type_arguments = cls.NumTypeArguments();
-  if (num_type_arguments > 0) {
+  if (!cls.HasTypeArguments()) {
     // No type arguments required for a non-parameterized type.
     ASSERT(Instance::CheckedHandle(arguments.At(1)).IsNull());
     return;
   }
   TypeArguments& type_arguments = TypeArguments::CheckedHandle(arguments.At(1));
   if (type_arguments.IsNull()) {
     // No instantiator is required for a raw type.
     ASSERT(Instance::CheckedHandle(arguments.At(2)).IsNull());
     return;
   }
-  ASSERT(type_arguments.Length() == num_type_arguments);
+  ASSERT(type_arguments.Length() == cls.NumTypeArguments());
   const TypeArguments& instantiator =
       TypeArguments::CheckedHandle(arguments.At(2));
   if (instantiator.IsNull()) {
     // Either the type argument vector is instantiated (use it), or the
     // instantiator is of a raw type and we cannot instantiate the type argument
     // vector (leave it as null).
     if (type_arguments.IsInstantiated()) {
       instance.SetTypeArguments(type_arguments);
     }
     return;
@@ skipping 834 matching lines @@
       }
     }
   }
   // The cache is null terminated, therefore the loop above should never
   // terminate by itself.
   UNREACHABLE();
   return Code::null();
 }
 
 }  // namespace dart