Implement Managed Detour in Il2CppDetourMethodPatcher (#237)

* implement IL Hook in Il2CppDetourMethodPatcher to skip managed->native->managed

* update to use detour instead of il hook

Author: XtraCube

Il2CppInterop.HarmonySupport/Il2CppDetourMethodPatcher.cs

@@ -12,9 +12,11 @@
 using Il2CppInterop.Runtime.Startup;
 using Microsoft.Extensions.Logging;
 using MonoMod.Cil;
+using MonoMod.RuntimeDetour;
 using MonoMod.Utils;
+using Detour = MonoMod.RuntimeDetour.Detour;
+using IDetour = Il2CppInterop.Runtime.Injection.IDetour;
 using ValueType = Il2CppSystem.ValueType;
-using Void = Il2CppSystem.Void;
 
 namespace Il2CppInterop.HarmonySupport;
 
@@ -57,6 +59,8 @@ private static readonly MethodInfo ReportExceptionMethodInfo
     };
 
     private static readonly List<object> DelegateCache = new();
+    private static readonly List<object> DetourCache = new();
+
     private INativeMethodInfoStruct modifiedNativeMethodInfo;
 
     private IDetour nativeDetour;
@@ -147,8 +151,10 @@ public override MethodBase DetourTo(MethodBase replacement)
         nativeDetour.Apply();
         modifiedNativeMethodInfo.MethodPointer = nativeDetour.OriginalTrampoline;
 
-        // TODO: Add an ILHook for the original unhollowed method to go directly to managedHookedMethod
-        // Right now it goes through three times as much interop conversion as it needs to, when being called from managed side
+        var detour = new Detour(Original, managedHookedMethod);
+        detour.Apply();
+        DetourCache.Add(detour);
+
         return managedHookedMethod;
     }