Remove getters that duplicate FunctionKind in SharedFunctionInfo and ParseInfo

src/compiler.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/compiler.h"
 
 #include <algorithm>
 #include <memory>
 
 #include "src/asmjs/asm-js.h"
@@ skipping 898 matching lines @@
   // We do not switch to baseline code when the debugger might have created a
   // copy of the bytecode with break slots to be able to set break points.
   if (function->shared()->HasDebugInfo()) {
     return MaybeHandle<Code>();
   }
 
   // TODO(4280): For now we do not switch generators or async functions to
   // baseline code because there might be suspended activations stored in
   // generator objects on the heap. We could eventually go directly to
   // TurboFan in this case.
-  if (function->shared()->is_resumable()) {
+  if (IsResumableFunction(function->shared()->kind())) {
     return MaybeHandle<Code>();
   }
 
   // TODO(4280): For now we disable switching to baseline code in the presence
   // of interpreter activations of the given function. The reasons is that the
   // underlying bytecode is cleared below. Note that this only applies in case
   // the --ignition-preserve-bytecode flag is not passed.
   if (!FLAG_ignition_preserve_bytecode) {
     InterpreterActivationsFinder activations_finder(function->shared());
     if (HasInterpreterActivations(isolate, &activations_finder)) {
@@ skipping 448 matching lines @@
   Handle<SharedFunctionInfo> shared = info->shared_info();
   if (!shared->has_deoptimization_support()) {
     Zone zone(info->isolate()->allocator());
     CompilationInfo unoptimized(info->parse_info(), info->closure());
     unoptimized.EnableDeoptimizationSupport();
 
     // TODO(4280): For now we do not switch generators or async functions to
     // baseline code because there might be suspended activations stored in
     // generator objects on the heap. We could eventually go directly to
     // TurboFan in this case.
-    if (shared->is_resumable()) return false;
+    if (IsResumableFunction(shared->kind())) return false;
 
     // TODO(4280): For now we disable switching to baseline code in the presence
     // of interpreter activations of the given function. The reasons is that the
     // underlying bytecode is cleared below. The expensive check for activations
     // only needs to be done when the given function has bytecode, otherwise we
     // can be sure there are no activations. Note that this only applies in case
     // the --ignition-preserve-bytecode flag is not passed.
     if (!FLAG_ignition_preserve_bytecode && shared->HasBytecodeArray()) {
       InterpreterActivationsFinder activations_finder(*shared);
       if (HasInterpreterActivations(info->isolate(), &activations_finder)) {
@@ skipping 514 matching lines @@
     DCHECK(shared->is_compiled());
     function->set_literals(cached.literals);
   } else if (shared->is_compiled()) {
     // TODO(mvstanton): pass pretenure flag to EnsureLiterals.
     JSFunction::EnsureLiterals(function);
   }
 }
 
 }  // namespace internal
 }  // namespace v8