Remove Assert in Method.MakeGeneric on Invalid Arg

The call to mono_class_inflate_generic_method_checked will set error
when there are invalid types (e.g. typeof(void)/typeof(int*)) passed in.
This should not be an assert, it should fall to the "Invalid generic
arguments" message below.   This does lose the error returned by
mono_class_inflate_generic_method_checked but that error isn't user
friendly.  It would report be something like: "MVAR 1 cannot be expanded
with type 0x1" Instead report the more readable error to the user.

Fix: 71339

Author: scott-ferguson-unity

src/mono/mono/metadata/reflection.c

@@ -2728,9 +2728,8 @@ reflection_bind_generic_method_parameters (MonoMethod *method, MonoArrayHandle t
 	tmp_context.method_inst = ginst;
 
 	inflated = mono_class_inflate_generic_method_checked (method, &tmp_context, error);
-	mono_error_assert_ok (error);
 
-	if (!mono_verifier_is_method_valid_generic_instantiation (inflated)) {
+	if (!is_ok(error) || !inflated || !mono_verifier_is_method_valid_generic_instantiation (inflated)) {
 		mono_error_set_argument (error, NULL, "Invalid generic arguments");
 		return NULL;
 	}

src/mono/out/build/x64-Debug/.cmake/api/v1/query/client-MicrosoftVS/query.json

@@ -0,0 +1 @@
+{"requests":[{"kind":"cache","version":2},{"kind":"cmakeFiles","version":1},{"kind":"codemodel","version":2},{"kind":"toolchains","version":1}]}

src/mono/out/build/x64-Debug/VSInheritEnvironments.txt

@@ -0,0 +1 @@
+msvc_x64_x64