[mono][interp] Disable inlining of methods that call methods using StackCrawlMark (#116134)

* [mono][interp] Don't inline callers of methods using StackCrawlMark

* [mono] Disable inlining of callers of Assembly.GetCallingAssembly

The stack mark is used only to prevent inlining of the callers of this method (since it sets a flag on the method in question). The stack mark itself is not currently used on mono.

Author: BrzVlad

src/mono/System.Private.CoreLib/src/System/Reflection/Assembly.Mono.cs

@@ -42,9 +42,15 @@ public static Assembly GetExecutingAssembly()
         [MethodImplAttribute(MethodImplOptions.InternalCall)]
         internal static extern RuntimeAssembly GetExecutingAssembly(ref StackCrawlMark stackMark);
 
-        [MethodImplAttribute(MethodImplOptions.InternalCall)]
         [System.Security.DynamicSecurityMethod] // Methods doing stack walks has to be marked DynamicSecurityMethod
-        public static extern Assembly GetCallingAssembly();
+        public static Assembly GetCallingAssembly()
+        {
+            StackCrawlMark stackMark = StackCrawlMark.LookForMyCallersCaller;
+            return GetCallingAssembly(ref stackMark);
+        }
+
+        [MethodImplAttribute(MethodImplOptions.InternalCall)]
+        internal static extern RuntimeAssembly GetCallingAssembly(ref StackCrawlMark stackMark);
 
         [MethodImplAttribute(MethodImplOptions.InternalCall)]
         internal static extern Assembly GetEntryAssemblyNative();

src/mono/mono/metadata/icall-def.h

@@ -284,7 +284,7 @@ ICALL_TYPE(OBJ, "System.Object", OBJ_3)
 HANDLES(OBJ_3, "MemberwiseClone", ves_icall_System_Object_MemberwiseClone, MonoObject, 1, (MonoObject))
 
 ICALL_TYPE(ASSEM, "System.Reflection.Assembly", ASSEM_2)
-HANDLES(ASSEM_2, "GetCallingAssembly", ves_icall_System_Reflection_Assembly_GetCallingAssembly, MonoReflectionAssembly, 0, ())
+HANDLES(ASSEM_2, "GetCallingAssembly", ves_icall_System_Reflection_Assembly_GetCallingAssembly, MonoReflectionAssembly, 1, (MonoStackCrawlMark_ptr))
 HANDLES(ASSEM_3, "GetEntryAssemblyNative", ves_icall_System_Reflection_Assembly_GetEntryAssembly, MonoReflectionAssembly, 0, ())
 HANDLES(ASSEM_4, "GetExecutingAssembly", ves_icall_System_Reflection_Assembly_GetExecutingAssembly, MonoReflectionAssembly, 1, (MonoStackCrawlMark_ptr))
 HANDLES(ASSEM_6, "InternalGetType", ves_icall_System_Reflection_Assembly_InternalGetType, MonoReflectionType, 5, (MonoReflectionAssembly, MonoReflectionModule, MonoString, MonoBoolean, MonoBoolean))

src/mono/mono/metadata/icall.c

@@ -5145,7 +5145,7 @@ ves_icall_System_Reflection_Assembly_GetEntryAssembly (MonoError *error)
 }
 
 MonoReflectionAssemblyHandle
-ves_icall_System_Reflection_Assembly_GetCallingAssembly (MonoError *error)
+ves_icall_System_Reflection_Assembly_GetCallingAssembly (MonoStackCrawlMark *stack_mark, MonoError *error)
 {
 	MonoMethod *m;
 	MonoMethod *dest;

src/mono/mono/mini/interp/transform.c

@@ -3849,6 +3849,8 @@ interp_transform_call (TransformData *td, MonoMethod *method, MonoMethod *target
 		} else if (td->method->wrapper_type == MONO_WRAPPER_RUNTIME_INVOKE) {
 			// This scenario causes https://github.com/dotnet/runtime/issues/83792
 			return FALSE;
+		} else if (target_method->flags & METHOD_ATTRIBUTE_REQSECOBJ) {
+			return FALSE;
 		} else if (has_doesnotreturn_attribute(target_method)) {
 			/*
 			 * Since the method does not return, it's probably a throw helper and will not be called.