[interp] Let compiler decide clear_resume_state inlining. (mono/mono#16704)
authorJay Krell <jaykrell@microsoft.com>
Mon, 9 Sep 2019 17:31:37 +0000 (10:31 -0700)
committerLarry Ewing <lewing@xamarin.com>
Mon, 9 Sep 2019 17:31:37 +0000 (12:31 -0500)
Manual inlining shows no additional stack use any longer.

Commit migrated from https://github.com/mono/mono/commit/83f965eca5cda8dc8bfd01b969a1d1481faa6977

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

index 4fd448d..d3d53f8 100644 (file)
@@ -228,9 +228,7 @@ int mono_interp_traceopt = 0;
 #define MINT_IN_DEFAULT default:
 #endif
 
-// FIXME The inlining of this needs to be reevaluated in the context of later changes.
-// Also there is now only one caller, so consider inlining it manually.
-static MONO_NEVER_INLINE GSList* // Inlining this causes caller to use more stack.
+static GSList*
 clear_resume_state (ThreadContext *context, InterpFrame *frame, GSList* finally_ips)
 {
        /* We have thrown an exception from a finally block. Some of the leave targets were unwound already */