[mono][interp] Disable inlining of methods that always throw (#122587)

When inlining, we create a placeholder exit_bb that is branched to
whenever the inlined method returns. When this bblock was not reachable,
it was leading to some inconsistent CFG state. This is relatively easy
to fix but, if exit_bb is dead, it means that the method is always
throwing an exception. We shouldn't bother inlining such methods in the
first place, since they are not part of hot code paths.

Fixes #122529

Author: BrzVlad

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

@@ -8888,10 +8888,10 @@ generate_code (TransformData *td, MonoMethod *method, MonoMethodHeader *header,
 				memcpy (td->stack, exit_bb->stack_state, exit_bb->stack_height * sizeof(td->stack [0]));
 			td->sp = td->stack + exit_bb->stack_height;
 		}
-		// If exit_bb is not reached by any other bb in this method, just mark it as dead so the
-		// method that does the inlining no longer generates code for the following IL opcodes.
+		// If exit_bb is not reached by any other bb in this method, it means the method always
+		// throws an exception. Inlining it isn't beneficial since this shouldn't be perf sensitive code.
 		if (exit_bb->in_count == 0)
-			exit_bb->dead = TRUE;
+			INLINE_FAILURE;
 		else
 			exit_bb->emit_state = BB_STATE_EMITTING;
 	}