Merge pull request #244 from tannergooding/main

Make sure JIT tests free allocated functions

Author: tannergooding

Directory.Packages.props

@@ -13,9 +13,9 @@
   <!-- Package versions for package references across all projects -->
   <ItemGroup>
     <PackageVersion Include="libLLVM" Version="20.1.2" />
-    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.9.0" />
-    <PackageVersion Include="NUnit" Version="4.1.0" />
-    <PackageVersion Include="NUnit3TestAdapter" Version="4.5.0" />
+    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.14.1" />
+    <PackageVersion Include="NUnit" Version="4.4.0" />
+    <PackageVersion Include="NUnit3TestAdapter" Version="5.1.0" />
   </ItemGroup>
 
 </Project>

tests/LLVMSharp.UnitTests/IR.cs

@@ -31,10 +31,13 @@ public void AddsSigned()
         _ = LLVM.InitializeNativeAsmParser();
         _ = LLVM.InitializeNativeAsmPrinter();
 
-        var engine = module.CreateMCJITCompiler();
+        using var engine = module.CreateMCJITCompiler();
         var func = engine.GetPointerToGlobal<Int32Int32Int32Delegate>(def);
         var result = op1 + op2;
         Assert.That(func(op1, op2), Is.EqualTo(result));
+
+        engine.FreeMachineCodeForFunction(def);
+        _ = engine.RemoveModule(module);
     }
 
     [Test]
@@ -56,10 +59,13 @@ public void ShiftsRight([Range(0, 256)] int op1, [Range(0, 8)] int op2)
         _ = LLVM.InitializeNativeAsmParser();
         _ = LLVM.InitializeNativeAsmPrinter();
 
-        var engine = module.CreateMCJITCompiler();
+        using var engine = module.CreateMCJITCompiler();
         var func = engine.GetPointerToGlobal<Int32Int32Int32Delegate>(def);
         var result = op1 >> op2;
         Assert.That(func(op1, op2), Is.EqualTo(result));
+
+        engine.FreeMachineCodeForFunction(def);
+        _ = engine.RemoveModule(module);
     }
 
     [Test]
@@ -81,10 +87,13 @@ public void ComparesGreaterThan([Range(0, 10)] int op1, [Range(0, 10)] int op2)
         _ = LLVM.InitializeNativeAsmParser();
         _ = LLVM.InitializeNativeAsmPrinter();
 
-        var engine = module.CreateMCJITCompiler();
+        using var engine = module.CreateMCJITCompiler();
         var func = engine.GetPointerToGlobal<Int32Int32Int8Delegate>(def);
         var result = op1 > op2 ? 1 : 0;
         Assert.That(func(op1, op2), Is.EqualTo(result));
+
+        engine.FreeMachineCodeForFunction(def);
+        _ = engine.RemoveModule(module);
     }
 
     [Test]
@@ -112,10 +121,14 @@ public void CallsFunction([Range(0, 10)] int op1, [Range(0, 10)] int op2)
         _ = LLVM.InitializeNativeAsmParser();
         _ = LLVM.InitializeNativeAsmPrinter();
 
-        var engine = module.CreateMCJITCompiler();
+        using var engine = module.CreateMCJITCompiler();
         var func = engine.GetPointerToGlobal<Int32Int32Int32Delegate>(entryDef);
         var result = op1 + op2;
         Assert.That(func(op1, op2), Is.EqualTo(result));
+
+        engine.FreeMachineCodeForFunction(entryDef);
+        engine.FreeMachineCodeForFunction(addDef);
+        _ = engine.RemoveModule(module);
     }
 
     [Test]
@@ -136,9 +149,12 @@ public void ReturnsConstant([Range(0, 10)] int input)
         _ = LLVM.InitializeNativeAsmParser();
         _ = LLVM.InitializeNativeAsmPrinter();
 
-        var engine = module.CreateMCJITCompiler();
+        using var engine = module.CreateMCJITCompiler();
         var func = engine.GetPointerToGlobal<Int32Delegate>(def);
         Assert.That(func(), Is.EqualTo(input));
+
+        engine.FreeMachineCodeForFunction(def);
+        _ = engine.RemoveModule(module);
     }
 
     [Test]
@@ -160,8 +176,11 @@ public void ReturnsSizeOf()
         _ = LLVM.InitializeNativeAsmParser();
         _ = LLVM.InitializeNativeAsmPrinter();
 
-        var engine = module.CreateMCJITCompiler();
+        using var engine = module.CreateMCJITCompiler();
         var func = engine.GetPointerToGlobal<Int32Delegate>(def);
         Assert.That(func(), Is.EqualTo(8));
+
+        engine.FreeMachineCodeForFunction(def);
+        _ = engine.RemoveModule(module);
     }
 }