Update comment from b60a2b93

src/ExpressionEvaluator/Core/Source/ExpressionCompiler/DkmUtilities.cs

@@ -21,6 +21,7 @@ internal static class DkmUtilities
 
         internal unsafe delegate IntPtr GetMetadataBytesPtrFunction(AssemblyIdentity assemblyIdentity, out uint uSize);
 
+        // Return the set of managed module instances from the AppDomain.
         private static IEnumerable<DkmClrModuleInstance> GetModulesInAppDomain(this DkmClrRuntimeInstance runtime, DkmClrAppDomain appDomain)
         {
             if (appDomain.IsUnloaded)
@@ -29,11 +30,11 @@ private static IEnumerable<DkmClrModuleInstance> GetModulesInAppDomain(this DkmC
             }
 
             var appDomainId = appDomain.Id;
+            // GetModuleInstances() may include instances of DkmClrNcContainerModuleInstance
+            // which are containers of managed module instances (see GetEmbeddedModules())
+            // but not managed modules themselves. Since GetModuleInstances() will include the
+            // embedded modules, we can simply ignore DkmClrNcContainerModuleInstances.
             return runtime.GetModuleInstances().
-                // Ignore module instances that are not DkmClrModuleInstance.
-                // Specifically, ignore DkmClrNcContainerModuleInstance.
-                // (The embedded DkmClrNcModuleInstance modules will
-                // be included in GetModuleInstances and will be used.)
                 OfType<DkmClrModuleInstance>().
                 Where(module =>
                 {