[vm,dyn_modules] Make sure interpreted NoSuchMethod dispatchers receive arguments descriptor

Fix Function::PrologueNeedsArgumentsDescriptor() to return true for
the interpreted NoSuchMethodDispatchers (although they have saved
arguments descriptor) as entering interpreter through InterpretCall
stub needs it.

This is needed to avoid switching instance calls to monomorphic state
if target is an interpreted NoSuchMethodDispatcher as monomorphic call
does not pass arguments descriptor and it is incompatible with
InterpretCall stub.

TEST=ci (co19/LibTest/convert/JsonCodec/encode_A02_t01)

Change-Id: I4675b27b827d36e3887d748412f5d582c5b43629
Cq-Include-Trybots: luci.dart.try:vm-aot-dyn-linux-debug-x64-try,vm-aot-dyn-linux-product-x64-try,vm-dyn-linux-debug-x64-try
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/445363
Commit-Queue: Alexander Markov <[email protected]>
Reviewed-by: Tess Strickland <[email protected]>

Author: alexmarkov

runtime/vm/object.cc

@@ -11997,17 +11997,17 @@ bool Function::HasDynamicCallers(Zone* zone) const {
 }
 
 bool Function::PrologueNeedsArgumentsDescriptor() const {
-  // These functions have a saved compile-time arguments descriptor that is
-  // used in lieu of the runtime arguments descriptor in generated IL.
-  if (HasSavedArgumentsDescriptor()) {
-    return false;
-  }
 #if defined(DART_DYNAMIC_MODULES)
   // Entering interpreter needs arguments descriptor.
   if (is_declared_in_bytecode()) {
     return true;
   }
 #endif
+  // These functions have a saved compile-time arguments descriptor that is
+  // used in lieu of the runtime arguments descriptor in generated IL.
+  if (HasSavedArgumentsDescriptor()) {
+    return false;
+  }
   // The prologue of those functions need to examine the arg descriptor for
   // various purposes.
   return IsGeneric() || HasOptionalParameters();