Ensure reflection has access to instance methods on byref-like types (#5165)

* Ensure reflection has access to instance methods on byref-like types

Adding the test uncovered a subtle issue. When rooting a virtual method for reflection, we should root the virtual method use of the slot defining method (not e.g. the `ToString` method on the byref-like type - we don't have virtual method use information for that).

* Disable part that doesn't work on Unix

1 files changed, 45 insertions, 0 deletions

diff --git a/tests/src/Simple/Reflection/Reflection.cs b/tests/src/Simple/Reflection/Reflection.cs
index 189090b3a..f24a855a8 100644
--- a/tests/src/Simple/Reflection/Reflection.cs
+++ b/tests/src/Simple/Reflection/Reflection.cs
@@ -28,6 +28,7 @@ internal class ReflectionTest
 #if !OPTIMIZED_MODE_WITHOUT_SCANNER
         TestContainment.Run();
         TestInterfaceMethod.Run();
+        TestByRefLikeTypeMethod.Run();
 #endif
         TestAttributeInheritance.Run();
         TestStringConstructor.Run();
@@ -324,6 +325,50 @@ internal class ReflectionTest
         }
     }
 
+    class TestByRefLikeTypeMethod
+    {
+        ref struct ByRefLike
+        {
+            public readonly int Value;
+
+            public ByRefLike(int value)
+            {
+                Value = value;
+            }
+
+            public override string ToString()
+            {
+                return Value.ToString();
+            }
+        }
+
+        delegate string ToStringDelegate(ref ByRefLike thisObj);
+
+        public static void Run()
+        {
+            Console.WriteLine(nameof(TestByRefLikeTypeMethod));
+
+            // Ensure things we reflect on are in the static callgraph
+            if (string.Empty.Length > 0)
+            {
+                default(ByRefLike).ToString();
+                ToStringDelegate s = null;
+                s = s.Invoke;
+            }
+
+            Type byRefLikeType = GetTestType(nameof(TestByRefLikeTypeMethod), nameof(ByRefLike));
+            MethodInfo toStringMethod = byRefLikeType.GetMethod("ToString");
+            /*var toString = (ToStringDelegate)*/toStringMethod.CreateDelegate(typeof(ToStringDelegate));
+
+            // Doesn't work on Unix: https://github.com/dotnet/corert/issues/4379
+#if false
+            ByRefLike foo = new ByRefLike(123);
+            if (toString(ref foo) != "123")
+                throw new Exception();
+#endif
+        }
+    }
+
     class TestInterfaceMethod
     {
         interface IFoo