add comment

max-charlamb · max-charlamb · commit 57d8fbad6151 · 2025-10-23T09:48:44.000-04:00
diff --git a/src/SOS/SOS.Hosting/RuntimeWrapper.cs b/src/SOS/SOS.Hosting/RuntimeWrapper.cs
@@ -158,10 +158,13 @@ protected override void Destroy()
             }
             if (_dacHandle != IntPtr.Zero)
             {
+                // Previously, the DAC was freed here, but as we transition to the cDAC which uses NativeAOT,
+                // it is no longer possible to free the DAC library when it is using the shimmed cDAC.
                 _dacHandle = IntPtr.Zero;
             }
             if (_cdacHandle != IntPtr.Zero)
             {
+                // cDAC can not be freed because it is a NativeAOT dll.
                 _cdacHandle = IntPtr.Zero;
             }
             if (_dbiHandle != IntPtr.Zero)

Original file line number	Diff line number	Diff line change
`@@ -158,10 +158,13 @@ protected override void Destroy()`
`158`	`158`	`}`
`159`	`159`	`if (_dacHandle != IntPtr.Zero)`
`160`	`160`	`{`
	`161`	`+ // Previously, the DAC was freed here, but as we transition to the cDAC which uses NativeAOT,`
	`162`	`+ // it is no longer possible to free the DAC library when it is using the shimmed cDAC.`
`161`	`163`	`_dacHandle = IntPtr.Zero;`
`162`	`164`	`}`
`163`	`165`	`if (_cdacHandle != IntPtr.Zero)`
`164`	`166`	`{`
	`167`	`+ // cDAC can not be freed because it is a NativeAOT dll.`
`165`	`168`	`_cdacHandle = IntPtr.Zero;`
`166`	`169`	`}`
`167`	`170`	`if (_dbiHandle != IntPtr.Zero)`