Only disable optmizations of -optimize is false. Not when any symbols…

… are emitted.
ikvmnet · Dec 12, 2024 · a2cc374 · a2cc374
1 parent 0389c9a
commit a2cc374
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/src/IKVM.Tools.Importer/ImportClassLoader.cs b/src/IKVM.Tools.Importer/ImportClassLoader.cs
@@ -171,16 +171,12 @@ internal ModuleBuilder CreateModuleBuilder()
             if (EmitStackTraceInfo)
                 Context.AttributeHelper.SetSourceFile(moduleBuilder, null);
 
-            // if configured to emit debug info or stack trace info, add debug DebuggableAttribute
-            if (EmitSymbols || EmitStackTraceInfo)
-            {
-                var debugModes = DebuggableAttribute.DebuggingModes.Default | DebuggableAttribute.DebuggingModes.IgnoreSymbolStoreSequencePoints;
-                if (EmitSymbols)
-                    debugModes |= DebuggableAttribute.DebuggingModes.DisableOptimizations;
-
-                Context.AttributeHelper.SetDebuggingModes(assemblyBuilder, debugModes);
-            }
+            // latest roslyn emits ignore symbol store always, but only disables optimizations if specified through args
+            var debugModes = DebuggableAttribute.DebuggingModes.Default | DebuggableAttribute.DebuggingModes.IgnoreSymbolStoreSequencePoints;
+            if (EnableOptimizations == false)
+                debugModes |= DebuggableAttribute.DebuggingModes.DisableOptimizations;
 
+            Context.AttributeHelper.SetDebuggingModes(assemblyBuilder, debugModes);
             Context.AttributeHelper.SetRuntimeCompatibilityAttribute(assemblyBuilder);
 
             if (state.baseAddress != 0)