[vm] Disable unchecked entrypoints to implicit closures in precompiled mode.

The check which disables entrypoints in precompiled mode currently only triggers for
regular functions. Here we make it trigger for implicit closures as well.

Change-Id: I68749a2109bb24ee5d299e4b139e3b63e3be5f72
Reviewed-on: https://dart-review.googlesource.com/70340
Reviewed-by: Alexander Markov <[email protected]>
Commit-Queue: Samir Jindel <[email protected]>

Author: sjindel-google

runtime/vm/compiler/frontend/kernel_binary_flowgraph.cc

@@ -1709,10 +1709,7 @@ StreamingFlowGraphBuilder::ChooseEntryPointStyle(
     const Fragment& every_time_prologue) {
   ASSERT(!dart_function.IsImplicitClosureFunction());
   if (!dart_function.MayHaveUncheckedEntryPoint(I) ||
-      implicit_type_checks.is_empty() ||
-      // TODO(#34162): Enable multiple-entrypoints for AOT if we
-      // consider them beneficial.
-      FLAG_precompiled_mode) {
+      implicit_type_checks.is_empty()) {
     return UncheckedEntryPointStyle::kNone;
   }
 

runtime/vm/object.cc

@@ -8296,8 +8296,10 @@ RawCode* Function::EnsureHasCode() const {
 
 bool Function::MayHaveUncheckedEntryPoint(Isolate* I) const {
 // TODO(#34162): Support the other architectures.
+// TODO(#34162): Enable multiple-entrypoints for AOT if we
+// consider them beneficial.
 #if defined(TARGET_ARCH_X64) || defined(TARGET_ARCH_ARM)
-  return FLAG_enable_multiple_entrypoints &&
+  return !FLAG_precompiled_mode && FLAG_enable_multiple_entrypoints &&
          (NeedsArgumentTypeChecks(I) || IsImplicitClosureFunction());
 #else
   return false;