diff --git a/src/linker/BannedSymbols.txt b/src/linker/BannedSymbols.txt index c39f451eee45..38bf6068a27b 100644 --- a/src/linker/BannedSymbols.txt +++ b/src/linker/BannedSymbols.txt @@ -1,2 +1,6 @@ T:Mono.Cecil.Cil.ILProcessor;Use LinkerILProcessor instead M:Mono.Cecil.TypeReference.Resolve();Use LinkContext.Resolve and LinkContext.TryResolve helpers instead +P:Mono.Cecil.Cil.MethodBody.Instructions;Use LinkContext.MethodBodyInstructionProvider instead +P:Mono.Cecil.Cil.MethodBody.ExceptionHandlers;Use LinkContext.MethodBodyInstructionProvider instead +P:Mono.Cecil.Cil.MethodBody.Variables;Use LinkContext.MethodBodyInstructionProvider instead +M:Mono.Linker.Steps.ILProvider/MethodIL.Create;Use ILProvider GetMethodIL instead \ No newline at end of file diff --git a/src/linker/Linker.Dataflow/CompilerGeneratedState.cs b/src/linker/Linker.Dataflow/CompilerGeneratedState.cs index 3213f49fdc49..0d240d4f1712 100644 --- a/src/linker/Linker.Dataflow/CompilerGeneratedState.cs +++ b/src/linker/Linker.Dataflow/CompilerGeneratedState.cs @@ -139,7 +139,7 @@ void ProcessMethod (MethodDefinition method) // Discover calls or references to lambdas or local functions. This includes // calls to local functions, and lambda assignments (which use ldftn). if (method.Body != null) { - foreach (var instruction in method.Body.Instructions) { + foreach (var instruction in _context.GetMethodIL (method).Instructions) { switch (instruction.OpCode.OperandType) { case OperandType.InlineMethod: { MethodDefinition? referencedMethod = _context.TryResolve ((MethodReference) instruction.Operand); @@ -348,7 +348,7 @@ void MapGeneratedTypeTypeParameters (TypeDefinition generatedType) GenericInstanceType? ScanForInit (TypeDefinition compilerGeneratedType, MethodBody body) { - foreach (var instr in body.Instructions) { + foreach (var instr in _context.GetMethodIL (body).Instructions) { bool handled = false; switch (instr.OpCode.Code) { case Code.Initobj: diff --git a/src/linker/Linker.Dataflow/FlowAnnotations.cs b/src/linker/Linker.Dataflow/FlowAnnotations.cs index 472f8567f063..5bde528568be 100644 --- a/src/linker/Linker.Dataflow/FlowAnnotations.cs +++ b/src/linker/Linker.Dataflow/FlowAnnotations.cs @@ -422,7 +422,7 @@ bool ScanMethodBodyForFieldAccess (MethodBody body, bool write, out FieldDefinit FieldReference? foundReference = null; - foreach (Instruction instruction in body.Instructions) { + foreach (Instruction instruction in _context.GetMethodIL (body).Instructions) { switch (instruction.OpCode.Code) { case Code.Ldsfld when !write: case Code.Ldfld when !write: diff --git a/src/linker/Linker.Dataflow/InterproceduralState.cs b/src/linker/Linker.Dataflow/InterproceduralState.cs index 09bc3be22630..7ef85d46fa61 100644 --- a/src/linker/Linker.Dataflow/InterproceduralState.cs +++ b/src/linker/Linker.Dataflow/InterproceduralState.cs @@ -4,6 +4,7 @@ using System; using System.Collections.Generic; using System.Diagnostics; +using ILLink.Shared; using ILLink.Shared.DataFlow; using Mono.Cecil; using Mono.Cecil.Cil; @@ -14,23 +15,25 @@ namespace Mono.Linker.Dataflow { - // Wrapper that implements IEquatable for MethodBody. - readonly record struct MethodBodyValue (MethodBody MethodBody); - // Tracks the set of methods which get analyzer together during interprocedural analysis, // and the possible states of hoisted locals in state machine methods and lambdas/local functions. struct InterproceduralState : IEquatable { - public ValueSet MethodBodies; + public ValueSet MethodBodies; public HoistedLocalState HoistedLocals; readonly InterproceduralStateLattice lattice; - public InterproceduralState (ValueSet methodBodies, HoistedLocalState hoistedLocals, InterproceduralStateLattice lattice) + public InterproceduralState (ValueSet methodBodies, HoistedLocalState hoistedLocals, InterproceduralStateLattice lattice) => (MethodBodies, HoistedLocals, this.lattice) = (methodBodies, hoistedLocals, lattice); public bool Equals (InterproceduralState other) => MethodBodies.Equals (other.MethodBodies) && HoistedLocals.Equals (other.HoistedLocals); + public override bool Equals (object? obj) + => obj is InterproceduralState state && Equals (state); + + public override int GetHashCode () => HashUtils.Combine(MethodBodies.GetHashCode (), HoistedLocals.GetHashCode ()); + public InterproceduralState Clone () => new (MethodBodies.Clone (), HoistedLocals.Clone (), lattice); @@ -43,23 +46,28 @@ public void TrackMethod (MethodDefinition method) } public void TrackMethod (MethodBody methodBody) + { + TrackMethod (lattice.Context.GetMethodIL (methodBody)); + } + + public void TrackMethod (MethodIL methodIL) { // Work around the fact that ValueSet is readonly - var methodsList = new List (MethodBodies); - methodsList.Add (new MethodBodyValue (methodBody)); + var methodsList = new List (MethodBodies); + methodsList.Add (methodIL); // For state machine methods, also scan the state machine members. // Simplification: assume that all generated methods of the state machine type are // reached at the point where the state machine method is reached. - if (CompilerGeneratedState.TryGetStateMachineType (methodBody.Method, out TypeDefinition? stateMachineType)) { + if (CompilerGeneratedState.TryGetStateMachineType (methodIL.Method, out TypeDefinition? stateMachineType)) { foreach (var stateMachineMethod in stateMachineType.Methods) { Debug.Assert (!CompilerGeneratedNames.IsLambdaOrLocalFunction (stateMachineMethod.Name)); if (stateMachineMethod.Body is MethodBody stateMachineMethodBody) - methodsList.Add (new MethodBodyValue (stateMachineMethodBody)); + methodsList.Add (lattice.Context.GetMethodIL (stateMachineMethodBody)); } } - MethodBodies = new ValueSet (methodsList); + MethodBodies = new ValueSet (methodsList); } public void SetHoistedLocal (HoistedLocalKey key, MultiValue value) @@ -76,15 +84,17 @@ public MultiValue GetHoistedLocal (HoistedLocalKey key) => HoistedLocals.Get (key); } - struct InterproceduralStateLattice : ILattice + readonly struct InterproceduralStateLattice : ILattice { - public readonly ValueSetLattice MethodBodyLattice; + public readonly ValueSetLattice MethodBodyLattice; public readonly DictionaryLattice> HoistedLocalsLattice; + public readonly LinkContext Context; public InterproceduralStateLattice ( - ValueSetLattice methodBodyLattice, - DictionaryLattice> hoistedLocalsLattice) - => (MethodBodyLattice, HoistedLocalsLattice) = (methodBodyLattice, hoistedLocalsLattice); + ValueSetLattice methodBodyLattice, + DictionaryLattice> hoistedLocalsLattice, + LinkContext context) + => (MethodBodyLattice, HoistedLocalsLattice, Context) = (methodBodyLattice, hoistedLocalsLattice, context); public InterproceduralState Top => new InterproceduralState (MethodBodyLattice.Top, HoistedLocalsLattice.Top, this); diff --git a/src/linker/Linker.Dataflow/MethodBodyScanner.cs b/src/linker/Linker.Dataflow/MethodBodyScanner.cs index ea5ef4dcbf0c..dad433b923f6 100644 --- a/src/linker/Linker.Dataflow/MethodBodyScanner.cs +++ b/src/linker/Linker.Dataflow/MethodBodyScanner.cs @@ -51,7 +51,7 @@ abstract partial class MethodBodyScanner protected MethodBodyScanner (LinkContext context) { this._context = context; - this.InterproceduralStateLattice = default; + this.InterproceduralStateLattice = new InterproceduralStateLattice (default, default, context); } internal MultiValue ReturnValue { private set; get; } @@ -151,9 +151,9 @@ private struct BasicBlockIterator int _currentBlockIndex; bool _foundEndOfPrevBlock; - public BasicBlockIterator (MethodBody methodBody) + public BasicBlockIterator (MethodIL methodIL) { - _methodBranchTargets = methodBody.ComputeBranchTargets (); + _methodBranchTargets = methodIL.ComputeBranchTargets (); _currentBlockIndex = -1; _foundEndOfPrevBlock = true; } @@ -226,9 +226,9 @@ protected static void StoreMethodLocalValue ( // Scans the method as well as any nested functions (local functions or lambdas) and state machines // reachable from it. - public virtual void InterproceduralScan (MethodBody startingMethodBody) + public virtual void InterproceduralScan (MethodIL startingMethodIL) { - MethodDefinition startingMethod = startingMethodBody.Method; + MethodDefinition startingMethod = startingMethodIL.Method; // Note that the default value of a hoisted local will be MultiValueLattice.Top, not UnknownValue.Instance. // This ensures that there are no warnings for the "unassigned state" of a parameter. @@ -236,15 +236,15 @@ public virtual void InterproceduralScan (MethodBody startingMethodBody) var interproceduralState = InterproceduralStateLattice.Top; var oldInterproceduralState = interproceduralState.Clone (); - interproceduralState.TrackMethod (startingMethodBody); + interproceduralState.TrackMethod (startingMethodIL); while (!interproceduralState.Equals (oldInterproceduralState)) { oldInterproceduralState = interproceduralState.Clone (); // Flow state through all methods encountered so far, as long as there // are changes discovered in the hoisted local state on entry to any method. - foreach (var methodBodyValue in oldInterproceduralState.MethodBodies) - Scan (methodBodyValue.MethodBody, ref interproceduralState); + foreach (var methodIL in oldInterproceduralState.MethodBodies) + Scan (methodIL, ref interproceduralState); } #if DEBUG @@ -274,22 +274,23 @@ void TrackNestedFunctionReference (MethodReference referencedMethod, ref Interpr interproceduralState.TrackMethod (method); } - protected virtual void Scan (MethodBody methodBody, ref InterproceduralState interproceduralState) + protected virtual void Scan (MethodIL methodIL, ref InterproceduralState interproceduralState) { + MethodBody methodBody = methodIL.Body; MethodDefinition thisMethod = methodBody.Method; - LocalVariableStore locals = new (methodBody.Variables.Count); + LocalVariableStore locals = new (methodIL.Variables.Count); Dictionary> knownStacks = new Dictionary> (); Stack? currentStack = new Stack (methodBody.MaxStackSize); - ScanExceptionInformation (knownStacks, methodBody); + ScanExceptionInformation (knownStacks, methodIL); - BasicBlockIterator blockIterator = new BasicBlockIterator (methodBody); + BasicBlockIterator blockIterator = new BasicBlockIterator (methodIL); ReturnValue = new (); - foreach (Instruction operation in methodBody.Instructions) { - ValidateNoReferenceToReference (locals, methodBody.Method, operation.Offset); + foreach (Instruction operation in methodIL.Instructions) { + ValidateNoReferenceToReference (locals, methodBody.Method, operation.Offset); int curBasicBlock = blockIterator.MoveNext (operation); if (knownStacks.ContainsKey (operation.Offset)) { @@ -403,7 +404,7 @@ protected virtual void Scan (MethodBody methodBody, ref InterproceduralState int case Code.Ldarg_S: case Code.Ldarga: case Code.Ldarga_S: - ScanLdarg (operation, currentStack, thisMethod, methodBody); + ScanLdarg (operation, currentStack, thisMethod, methodIL); break; case Code.Ldloc: @@ -414,7 +415,7 @@ protected virtual void Scan (MethodBody methodBody, ref InterproceduralState int case Code.Ldloc_S: case Code.Ldloca: case Code.Ldloca_S: - ScanLdloc (operation, currentStack, methodBody, locals); + ScanLdloc (operation, currentStack, methodIL, locals); break; case Code.Ldstr: { @@ -577,7 +578,7 @@ protected virtual void Scan (MethodBody methodBody, ref InterproceduralState int case Code.Stloc_1: case Code.Stloc_2: case Code.Stloc_3: - ScanStloc (operation, currentStack, methodBody, locals, curBasicBlock); + ScanStloc (operation, currentStack, methodIL, locals, curBasicBlock); break; case Code.Constrained: @@ -697,9 +698,9 @@ protected virtual void Scan (MethodBody methodBody, ref InterproceduralState int } } - private static void ScanExceptionInformation (Dictionary> knownStacks, MethodBody methodBody) + private static void ScanExceptionInformation (Dictionary> knownStacks, MethodIL methodIL) { - foreach (ExceptionHandler exceptionClause in methodBody.ExceptionHandlers) { + foreach (ExceptionHandler exceptionClause in methodIL.ExceptionHandlers) { Stack catchStack = new Stack (1); catchStack.Push (new StackSlot ()); @@ -715,7 +716,7 @@ private static void ScanExceptionInformation (Dictionary> protected abstract SingleValue GetMethodParameterValue (MethodDefinition method, int parameterIndex); - private void ScanLdarg (Instruction operation, Stack currentStack, MethodDefinition thisMethod, MethodBody methodBody) + private void ScanLdarg (Instruction operation, Stack currentStack, MethodDefinition thisMethod, MethodIL methodBody) { Code code = operation.OpCode.Code; @@ -741,7 +742,7 @@ private void ScanLdarg (Instruction operation, Stack currentStack, Me } else { var paramDefinition = (ParameterDefinition) operation.Operand; if (thisMethod.HasImplicitThis ()) { - if (paramDefinition == methodBody.ThisParameter) { + if (paramDefinition == methodBody.Body.ThisParameter) { paramNum = 0; } else { paramNum = paramDefinition.Index + 1; @@ -781,12 +782,12 @@ private void ScanStarg ( private void ScanLdloc ( Instruction operation, Stack currentStack, - MethodBody methodBody, + MethodIL methodIL, LocalVariableStore locals) { - VariableDefinition localDef = GetLocalDef (operation, methodBody.Variables); + VariableDefinition localDef = GetLocalDef (operation, methodIL.Variables); if (localDef == null) { - PushUnknownAndWarnAboutInvalidIL (currentStack, methodBody, operation.Offset); + PushUnknownAndWarnAboutInvalidIL (currentStack, methodIL.Body, operation.Offset); return; } @@ -844,14 +845,14 @@ void ScanLdtoken (Instruction operation, Stack currentStack) private void ScanStloc ( Instruction operation, Stack currentStack, - MethodBody methodBody, + MethodIL methodIL, LocalVariableStore locals, int curBasicBlock) { - StackSlot valueToStore = PopUnknown (currentStack, 1, methodBody, operation.Offset); - VariableDefinition localDef = GetLocalDef (operation, methodBody.Variables); + StackSlot valueToStore = PopUnknown (currentStack, 1, methodIL.Body, operation.Offset); + VariableDefinition localDef = GetLocalDef (operation, methodIL.Variables); if (localDef == null) { - WarnAboutInvalidILInMethod (methodBody, operation.Offset); + WarnAboutInvalidILInMethod (methodIL.Body, operation.Offset); return; } diff --git a/src/linker/Linker.Dataflow/ReflectionMethodBodyScanner.cs b/src/linker/Linker.Dataflow/ReflectionMethodBodyScanner.cs index 56d5549894c1..a4d87fe639a4 100644 --- a/src/linker/Linker.Dataflow/ReflectionMethodBodyScanner.cs +++ b/src/linker/Linker.Dataflow/ReflectionMethodBodyScanner.cs @@ -61,21 +61,21 @@ public ReflectionMethodBodyScanner (LinkContext context, MarkStep parent, Messag TrimAnalysisPatterns = new TrimAnalysisPatternStore (MultiValueLattice, context); } - public override void InterproceduralScan (MethodBody methodBody) + public override void InterproceduralScan (MethodIL methodIL) { - base.InterproceduralScan (methodBody); + base.InterproceduralScan (methodIL); var reflectionMarker = new ReflectionMarker (_context, _markStep, enabled: true); TrimAnalysisPatterns.MarkAndProduceDiagnostics (reflectionMarker, _markStep); } - protected override void Scan (MethodBody methodBody, ref InterproceduralState interproceduralState) + protected override void Scan (MethodIL methodIL, ref InterproceduralState interproceduralState) { - _origin = new MessageOrigin (methodBody.Method); - base.Scan (methodBody, ref interproceduralState); + _origin = new MessageOrigin (methodIL.Method); + base.Scan (methodIL, ref interproceduralState); - if (!methodBody.Method.ReturnsVoid ()) { - var method = methodBody.Method; + if (!methodIL.Method.ReturnsVoid ()) { + var method = methodIL.Method; var methodReturnValue = _annotations.GetMethodReturnValue (method); if (methodReturnValue.DynamicallyAccessedMemberTypes != 0) HandleAssignmentPattern (_origin, ReturnValue, methodReturnValue); diff --git a/src/linker/Linker.Dataflow/ScannerExtensions.cs b/src/linker/Linker.Dataflow/ScannerExtensions.cs index c445f7d61c20..7b8dbba87eb6 100644 --- a/src/linker/Linker.Dataflow/ScannerExtensions.cs +++ b/src/linker/Linker.Dataflow/ScannerExtensions.cs @@ -16,10 +16,10 @@ public static bool IsControlFlowInstruction (in this OpCode opcode) || (opcode.FlowControl == FlowControl.Return && opcode.Code != Code.Ret); } - public static HashSet ComputeBranchTargets (this MethodBody methodBody) + public static HashSet ComputeBranchTargets (this MethodIL methodIL) { HashSet branchTargets = new HashSet (); - foreach (Instruction operation in methodBody.Instructions) { + foreach (Instruction operation in methodIL.Instructions) { if (!operation.OpCode.IsControlFlowInstruction ()) continue; Object value = operation.Operand; @@ -31,7 +31,7 @@ public static HashSet ComputeBranchTargets (this MethodBody methodBody) } } } - foreach (ExceptionHandler einfo in methodBody.ExceptionHandlers) { + foreach (ExceptionHandler einfo in methodIL.ExceptionHandlers) { if (einfo.HandlerType == ExceptionHandlerType.Filter) { branchTargets.Add (einfo.FilterStart.Offset); } diff --git a/src/linker/Linker.Steps/AddBypassNGenStep.cs b/src/linker/Linker.Steps/AddBypassNGenStep.cs index e8706e1d6cc2..9a6443173a5d 100644 --- a/src/linker/Linker.Steps/AddBypassNGenStep.cs +++ b/src/linker/Linker.Steps/AddBypassNGenStep.cs @@ -94,7 +94,9 @@ private void EnsureBypassNGenAttribute (ModuleDefinition targetModule) const MethodAttributes ctorAttributes = MethodAttributes.Public | MethodAttributes.HideBySig | MethodAttributes.SpecialName | MethodAttributes.RTSpecialName; bypassNGenAttributeDefaultConstructor = new MethodDefinition (".ctor", ctorAttributes, coreLibAssembly.MainModule.TypeSystem.Void); +#pragma warning disable RS0030 // Anything after MarkStep should use Cecil directly as all method bodies should be processed by this point var instructions = bypassNGenAttributeDefaultConstructor.Body.Instructions; +#pragma warning restore RS0030 instructions.Add (Instruction.Create (OpCodes.Ldarg_0)); instructions.Add (Instruction.Create (OpCodes.Call, systemAttributeDefaultConstructor)); instructions.Add (Instruction.Create (OpCodes.Ret)); diff --git a/src/linker/Linker.Steps/CodeRewriterStep.cs b/src/linker/Linker.Steps/CodeRewriterStep.cs index 716c1fd1ea8f..8ac81022ac7c 100644 --- a/src/linker/Linker.Steps/CodeRewriterStep.cs +++ b/src/linker/Linker.Steps/CodeRewriterStep.cs @@ -64,12 +64,15 @@ void AddFieldsInitializations (TypeDefinition type) ret = Instruction.Create (OpCodes.Ret); processor.Append (ret); } else { - ret = cctor.Body.Instructions.Last (l => l.OpCode.Code == Code.Ret); var body = cctor.Body; - processor = cctor.Body.GetLinkerILProcessor (); - - for (int i = 0; i < body.Instructions.Count; ++i) { - var instr = body.Instructions[i]; +#pragma warning disable RS0030 // After MarkStep all methods should be processed and thus accessing Cecil directly is the right approach + var instructions = body.Instructions; +#pragma warning restore RS0030 + ret = instructions.Last (l => l.OpCode.Code == Code.Ret); + processor = body.GetLinkerILProcessor (); + + for (int i = 0; i < instructions.Count; ++i) { + var instr = instructions[i]; if (instr.OpCode.Code != Code.Stsfld) continue; @@ -199,7 +202,9 @@ static void StubComplexBody (MethodDefinition method, MethodBody body, LinkerILP case MetadataType.MVar: case MetadataType.ValueType: var vd = new VariableDefinition (method.ReturnType); +#pragma warning disable RS0030 // Anything after MarkStep should not use ILProvider since all methods are guaranteed processed body.Variables.Add (vd); +#pragma warning restore RS0030 body.InitLocals = true; il.Emit (OpCodes.Ldloca_S, vd); diff --git a/src/linker/Linker.Steps/MarkStep.cs b/src/linker/Linker.Steps/MarkStep.cs index 7dbede11d0fd..88dc5bba99b6 100644 --- a/src/linker/Linker.Steps/MarkStep.cs +++ b/src/linker/Linker.Steps/MarkStep.cs @@ -75,13 +75,6 @@ protected LinkContext Context { // method body scanner. readonly Dictionary _compilerGeneratedMethodRequiresScanner; - UnreachableBlocksOptimizer? _unreachableBlocksOptimizer; - UnreachableBlocksOptimizer UnreachableBlocksOptimizer { - get { - Debug.Assert (_unreachableBlocksOptimizer != null); - return _unreachableBlocksOptimizer; - } - } MarkStepContext? _markContext; MarkStepContext MarkContext { get { @@ -245,7 +238,6 @@ public MarkStep () public virtual void Process (LinkContext context) { _context = context; - _unreachableBlocksOptimizer = new UnreachableBlocksOptimizer (_context); _markContext = new MarkStepContext (); _scopeStack = new MarkScopeStack (); _dynamicallyAccessedMembersTypeHierarchy = new DynamicallyAccessedMembersTypeHierarchy (_context, this); @@ -2553,7 +2545,7 @@ void MarkICustomMarshalerMethods (TypeDefinition inputType, in DependencyInfo re } while ((type = Context.TryResolve (type.BaseType)) != null); } - static bool IsNonEmptyStaticConstructor (MethodDefinition method) + bool IsNonEmptyStaticConstructor (MethodDefinition method) { if (!method.IsStaticConstructor ()) return false; @@ -2561,10 +2553,12 @@ static bool IsNonEmptyStaticConstructor (MethodDefinition method) if (!method.HasBody || !method.IsIL) return true; - if (method.Body.CodeSize != 1) + var body = Context.GetMethodIL (method); + + if (body.Body.CodeSize != 1) return true; - return method.Body.Instructions[0].OpCode.Code != Code.Ret; + return body.Instructions[0].OpCode.Code != Code.Ret; } static bool HasOnSerializeOrDeserializeAttribute (MethodDefinition method) @@ -2840,14 +2834,14 @@ protected bool MarkFields (TypeDefinition type, bool includeStatic, in Dependenc return true; } - static PropertyDefinition? SearchPropertiesForMatchingFieldDefinition (FieldDefinition field) + PropertyDefinition? SearchPropertiesForMatchingFieldDefinition (FieldDefinition field) { foreach (var property in field.DeclaringType.Properties) { - var instr = property.GetMethod?.Body?.Instructions; - if (instr == null) + var body = property.GetMethod?.Body; + if (body == null) continue; - foreach (var ins in instr) { + foreach (var ins in Context.GetMethodIL (body).Instructions) { if (ins?.Operand == field) return property; } @@ -2939,7 +2933,7 @@ bool ShouldWarnForReflectionAccessToCompilerGeneratedCode (MethodDefinition meth // the reflection scanner. Checking this will also mark direct dependencies of the method body, if it // hasn't been marked already. A cache ensures this only happens once for the method, whether or not // it is accessed via reflection. - return CheckRequiresReflectionMethodBodyScanner (method.Body); + return CheckRequiresReflectionMethodBodyScanner (Context.GetMethodIL (method)); } void ProcessAnalysisAnnotationsForMethod (MethodDefinition method, DependencyKind dependencyKind, in MessageOrigin origin) @@ -3121,8 +3115,6 @@ protected virtual void ProcessMethod (MethodDefinition method, in DependencyInfo if (CheckProcessed (method)) return; - UnreachableBlocksOptimizer.ProcessMethod (method); - foreach (Action handleMarkMethod in MarkContext.MarkMethodActions) handleMarkMethod (method); @@ -3474,7 +3466,9 @@ internal void MarkMethodIfNotNull (MethodReference method, in DependencyInfo rea protected virtual void MarkMethodBody (MethodBody body) { - if (Context.IsOptimizationEnabled (CodeOptimizations.UnreachableBodies, body.Method) && IsUnreachableBody (body)) { + var processedMethodBody = Context.GetMethodIL (body); + + if (Context.IsOptimizationEnabled (CodeOptimizations.UnreachableBodies, body.Method) && IsUnreachableBody (processedMethodBody)) { MarkAndCacheConvertToThrowExceptionCtor (new DependencyInfo (DependencyKind.UnreachableBodyRequirement, body.Method)); _unreachableBodies.Add ((body, ScopeStack.CurrentScope)); return; @@ -3485,24 +3479,24 @@ protected virtual void MarkMethodBody (MethodBody body) // But for compiler-generated methods we only do dataflow analysis if they're used through their // corresponding user method, so we will skip dataflow for compiler-generated methods which // are only accessed via reflection. - bool requiresReflectionMethodBodyScanner = MarkAndCheckRequiresReflectionMethodBodyScanner (body); + bool requiresReflectionMethodBodyScanner = MarkAndCheckRequiresReflectionMethodBodyScanner (processedMethodBody); // Data-flow (reflection scanning) for compiler-generated methods will happen as part of the // data-flow scan of the user-defined method which uses this compiler-generated method. if (CompilerGeneratedState.IsNestedFunctionOrStateMachineMember (body.Method)) return; - MarkReflectionLikeDependencies (body, requiresReflectionMethodBodyScanner); + MarkReflectionLikeDependencies (processedMethodBody, requiresReflectionMethodBodyScanner); } - bool CheckRequiresReflectionMethodBodyScanner (MethodBody body) + bool CheckRequiresReflectionMethodBodyScanner (MethodIL methodIL) { // This method is only called on reflection access to compiler-generated methods. // This should be uncommon, so don't cache the result. - if (ReflectionMethodBodyScanner.RequiresReflectionMethodBodyScannerForMethodBody (Context, body.Method)) + if (ReflectionMethodBodyScanner.RequiresReflectionMethodBodyScannerForMethodBody (Context, methodIL.Method)) return true; - foreach (Instruction instruction in body.Instructions) { + foreach (Instruction instruction in methodIL.Instructions) { switch (instruction.OpCode.OperandType) { case OperandType.InlineField: if (InstructionRequiresReflectionMethodBodyScannerForFieldAccess (instruction)) @@ -3522,64 +3516,64 @@ bool CheckRequiresReflectionMethodBodyScanner (MethodBody body) // It computes the same value, while also marking as it goes, as an optimization. // This should only be called behind a check to IsProcessed for the method or corresponding user method, // to avoid recursion. - bool MarkAndCheckRequiresReflectionMethodBodyScanner (MethodBody body) + bool MarkAndCheckRequiresReflectionMethodBodyScanner (MethodIL methodIL) { #if DEBUG - if (!Annotations.IsProcessed (body.Method)) { - Debug.Assert (CompilerGeneratedState.IsNestedFunctionOrStateMachineMember (body.Method)); - MethodDefinition owningMethod = body.Method; + if (!Annotations.IsProcessed (methodIL.Method)) { + Debug.Assert (CompilerGeneratedState.IsNestedFunctionOrStateMachineMember (methodIL.Method)); + MethodDefinition owningMethod = methodIL.Method; while (Context.CompilerGeneratedState.TryGetOwningMethodForCompilerGeneratedMember (owningMethod, out var owner)) owningMethod = owner; - Debug.Assert (owningMethod != body.Method); + Debug.Assert (owningMethod != methodIL.Method); Debug.Assert (Annotations.IsProcessed (owningMethod)); } #endif // This may get called multiple times for compiler-generated code: once for // reflection access, and once as part of the interprocedural scan of the user method. // This check ensures that we only do the work and produce warnings once. - if (_compilerGeneratedMethodRequiresScanner.TryGetValue (body, out bool requiresReflectionMethodBodyScanner)) + if (_compilerGeneratedMethodRequiresScanner.TryGetValue (methodIL.Body, out bool requiresReflectionMethodBodyScanner)) return requiresReflectionMethodBodyScanner; - foreach (VariableDefinition var in body.Variables) - MarkType (var.VariableType, new DependencyInfo (DependencyKind.VariableType, body.Method)); + foreach (VariableDefinition var in methodIL.Variables) + MarkType (var.VariableType, new DependencyInfo (DependencyKind.VariableType, methodIL.Method)); - foreach (ExceptionHandler eh in body.ExceptionHandlers) + foreach (ExceptionHandler eh in methodIL.ExceptionHandlers) if (eh.HandlerType == ExceptionHandlerType.Catch) - MarkType (eh.CatchType, new DependencyInfo (DependencyKind.CatchType, body.Method)); + MarkType (eh.CatchType, new DependencyInfo (DependencyKind.CatchType, methodIL.Method)); requiresReflectionMethodBodyScanner = - ReflectionMethodBodyScanner.RequiresReflectionMethodBodyScannerForMethodBody (Context, body.Method); - using var _ = ScopeStack.PushScope (new MessageOrigin (body.Method)); - foreach (Instruction instruction in body.Instructions) - MarkInstruction (instruction, body.Method, ref requiresReflectionMethodBodyScanner); + ReflectionMethodBodyScanner.RequiresReflectionMethodBodyScannerForMethodBody (Context, methodIL.Method); + using var _ = ScopeStack.PushScope (new MessageOrigin (methodIL.Method)); + foreach (Instruction instruction in methodIL.Instructions) + MarkInstruction (instruction, methodIL.Method, ref requiresReflectionMethodBodyScanner); - MarkInterfacesNeededByBodyStack (body); + MarkInterfacesNeededByBodyStack (methodIL); - if (CompilerGeneratedState.IsNestedFunctionOrStateMachineMember (body.Method)) - _compilerGeneratedMethodRequiresScanner.Add (body, requiresReflectionMethodBodyScanner); + if (CompilerGeneratedState.IsNestedFunctionOrStateMachineMember (methodIL.Method)) + _compilerGeneratedMethodRequiresScanner.Add (methodIL.Body, requiresReflectionMethodBodyScanner); - PostMarkMethodBody (body); + PostMarkMethodBody (methodIL.Body); - Debug.Assert (requiresReflectionMethodBodyScanner == CheckRequiresReflectionMethodBodyScanner (body)); + Debug.Assert (requiresReflectionMethodBodyScanner == CheckRequiresReflectionMethodBodyScanner (methodIL)); return requiresReflectionMethodBodyScanner; } - bool IsUnreachableBody (MethodBody body) + bool IsUnreachableBody (MethodIL methodIL) { - return !body.Method.IsStatic - && !Annotations.IsInstantiated (body.Method.DeclaringType) - && MethodBodyScanner.IsWorthConvertingToThrow (body); + return !methodIL.Method.IsStatic + && !Annotations.IsInstantiated (methodIL.Method.DeclaringType) + && MethodBodyScanner.IsWorthConvertingToThrow (methodIL); } partial void PostMarkMethodBody (MethodBody body); - void MarkInterfacesNeededByBodyStack (MethodBody body) + void MarkInterfacesNeededByBodyStack (MethodIL methodIL) { // If a type could be on the stack in the body and an interface it implements could be on the stack on the body // then we need to mark that interface implementation. When this occurs it is not safe to remove the interface implementation from the type // even if the type is never instantiated - var implementations = new InterfacesOnStackScanner (Context).GetReferencedInterfaces (body); + var implementations = new InterfacesOnStackScanner (Context).GetReferencedInterfaces (methodIL); if (implementations == null) return; @@ -3733,24 +3727,24 @@ protected internal virtual bool ProcessReflectionDependency (MethodBody body, In // // Tries to mark additional dependencies used in reflection like calls (e.g. typeof (MyClass).GetField ("fname")) // - protected virtual void MarkReflectionLikeDependencies (MethodBody body, bool requiresReflectionMethodBodyScanner) + protected virtual void MarkReflectionLikeDependencies (MethodIL methodIL, bool requiresReflectionMethodBodyScanner) { - Debug.Assert (!CompilerGeneratedState.IsNestedFunctionOrStateMachineMember (body.Method)); + Debug.Assert (!CompilerGeneratedState.IsNestedFunctionOrStateMachineMember (methodIL.Method)); // requiresReflectionMethodBodyScanner tells us whether the method body itself requires a dataflow scan. // If the method body owns any compiler-generated code, we might still need to do a scan of it together with // all of the compiler-generated code it owns, so first check any compiler-generated callees. - if (Context.CompilerGeneratedState.TryGetCompilerGeneratedCalleesForUserMethod (body.Method, out List? compilerGeneratedCallees)) { + if (Context.CompilerGeneratedState.TryGetCompilerGeneratedCalleesForUserMethod (methodIL.Method, out List? compilerGeneratedCallees)) { foreach (var compilerGeneratedCallee in compilerGeneratedCallees) { switch (compilerGeneratedCallee) { case MethodDefinition nestedFunction: if (nestedFunction.Body is MethodBody nestedBody) - requiresReflectionMethodBodyScanner |= MarkAndCheckRequiresReflectionMethodBodyScanner (nestedBody); + requiresReflectionMethodBodyScanner |= MarkAndCheckRequiresReflectionMethodBodyScanner (Context.GetMethodIL (nestedBody)); break; case TypeDefinition stateMachineType: foreach (var method in stateMachineType.Methods) { if (method.Body is MethodBody stateMachineBody) - requiresReflectionMethodBodyScanner |= MarkAndCheckRequiresReflectionMethodBodyScanner (stateMachineBody); + requiresReflectionMethodBodyScanner |= MarkAndCheckRequiresReflectionMethodBodyScanner (Context.GetMethodIL (stateMachineBody)); } break; default: @@ -3762,9 +3756,9 @@ protected virtual void MarkReflectionLikeDependencies (MethodBody body, bool req if (!requiresReflectionMethodBodyScanner) return; - Debug.Assert (ScopeStack.CurrentScope.Origin.Provider == body.Method); + Debug.Assert (ScopeStack.CurrentScope.Origin.Provider == methodIL.Method); var scanner = new ReflectionMethodBodyScanner (Context, this, ScopeStack.CurrentScope.Origin); - scanner.InterproceduralScan (body); + scanner.InterproceduralScan (methodIL); } protected class AttributeProviderPair diff --git a/src/linker/Linker.Steps/UnreachableBlocksOptimizer.cs b/src/linker/Linker.Steps/UnreachableBlocksOptimizer.cs index d91fb09dd920..14377cbf6a10 100644 --- a/src/linker/Linker.Steps/UnreachableBlocksOptimizer.cs +++ b/src/linker/Linker.Steps/UnreachableBlocksOptimizer.cs @@ -434,9 +434,11 @@ public bool RewriteBody () { bool changed = false; LinkerILProcessor processor = body.GetLinkerILProcessor (); +#pragma warning disable RS0030 // This optimizer is the reason for the banned API, so it needs to use the Cecil directly Collection instrs = body.Instructions; +#pragma warning restore RS0030 - for (int i = 0; i < body.Instructions.Count; ++i) { + for (int i = 0; i < instrs.Count; ++i) { Instruction instr = instrs[i]; switch (instr.OpCode.Code) { @@ -455,7 +457,7 @@ public bool RewriteBody () if (md.NoInlining) break; - var cpl = new CalleePayload (md, GetArgumentsOnStack (md, body.Instructions, i)); + var cpl = new CalleePayload (md, GetArgumentsOnStack (md, instrs, i)); MethodResult? call_result = optimizer.TryGetMethodCallResult (cpl); if (call_result is not MethodResult result) break; @@ -550,6 +552,11 @@ public BodyReducer (MethodBody body, LinkContext context) public MethodBody Body { get; } +#pragma warning disable RS0030 // This optimizer is the reason for the banned API, so it needs to use the Cecil directly + Collection Instructions => Body.Instructions; + Collection ExceptionHandlers => Body.ExceptionHandlers; +#pragma warning restore RS0030 + public int InstructionsReplaced { get; set; } Collection? FoldedInstructions { get; set; } @@ -558,7 +565,7 @@ public BodyReducer (MethodBody body, LinkContext context) [MemberNotNull ("mapping")] void InitializeFoldedInstruction () { - FoldedInstructions = new Collection (Body.Instructions); + FoldedInstructions = new Collection (Instructions); mapping = new Dictionary (); } @@ -571,7 +578,7 @@ public void Rewrite (int index, Instruction newInstruction) // Tracks mapping for replaced instructions for easier // branch targets resolution later - mapping[Body.Instructions[index]] = index; + mapping[Instructions[index]] = index; FoldedInstructions[index] = newInstruction; } @@ -735,7 +742,7 @@ public bool RewriteBody () public bool ApplyTemporaryInlining (in UnreachableBlocksOptimizer optimizer) { bool changed = false; - var instructions = Body.Instructions; + var instructions = Instructions; Instruction? targetResult; for (int i = 0; i < instructions.Count; ++i) { @@ -826,10 +833,10 @@ void RemoveUnreachableInstructions (BitArray reachable) // inject "ldnull; throw;" at the end - this branch should never be reachable and it's always valid // (ret may need to return a value of the right type if the method has a return value which is complicated // to construct out of nothing). - if (index == Body.Instructions.Count - 1 && Body.Instructions[index].OpCode == OpCodes.Ret && - index > 0 && IsConditionalBranch (Body.Instructions[index - 1].OpCode)) { + if (index == Instructions.Count - 1 && Instructions[index].OpCode == OpCodes.Ret && + index > 0 && IsConditionalBranch (Instructions[index - 1].OpCode)) { processor.Replace (index, Instruction.Create (OpCodes.Ldnull)); - processor.InsertAfter (Body.Instructions[index], Instruction.Create (OpCodes.Throw)); + processor.InsertAfter (Instructions[index], Instruction.Create (OpCodes.Throw)); } else { processor.RemoveAt (index); ++removed; @@ -1046,8 +1053,8 @@ BitArray GetReachableInstructionsMap (out List? unreachableHan if (!exceptionHandlersChecked) { exceptionHandlersChecked = true; - var instrs = Body.Instructions; - foreach (var handler in Body.ExceptionHandlers) { + var instrs = Instructions; + foreach (var handler in ExceptionHandlers) { int start = instrs.IndexOf (handler.TryStart); int end = instrs.IndexOf (handler.TryEnd) - 1; @@ -1160,6 +1167,11 @@ bool IsJumpTargetRange (int firstInstr, int lastInstr) struct BodySweeper { readonly MethodBody body; +#pragma warning disable RS0030 // This optimizer is the reason for the banned API, so it needs to use the Cecil directly + Collection Instructions => body.Instructions; + Collection Variables => body.Variables; + Collection ExceptionHandlers => body.ExceptionHandlers; +#pragma warning restore RS0030 readonly BitArray reachable; readonly List? unreachableExceptionHandlers; readonly LinkContext context; @@ -1186,7 +1198,7 @@ public BodySweeper (MethodBody body, BitArray reachable, List? public void Initialize () { - var instrs = body.Instructions; + var instrs = Instructions; // // Reusing same reachable map and altering it at indexes @@ -1221,7 +1233,7 @@ public void Process (List? conditionInstrsToRemove, out List? // Initial pass which replaces unreachable instructions with nops or // ret to keep the body verifiable // - var instrs = body.Instructions; + var instrs = Instructions; for (int i = 0; i < instrs.Count; ++i) { if (reachable[i]) continue; @@ -1310,7 +1322,7 @@ public void Process (List? conditionInstrsToRemove, out List? void CleanRemovedVariables (List variables) { - foreach (var instr in body.Instructions) { + foreach (var instr in Instructions) { VariableDefinition? variable = GetVariableReference (instr); if (variable == null) continue; @@ -1323,7 +1335,7 @@ void CleanRemovedVariables (List variables) } variables.Sort ((a, b) => b.Index.CompareTo (a.Index)); - var body_variables = body.Variables; + var body_variables = Variables; foreach (var variable in variables) { var index = body_variables.IndexOf (variable); @@ -1348,7 +1360,7 @@ void CleanExceptionHandlers () return; foreach (var eh in unreachableExceptionHandlers) - body.ExceptionHandlers.Remove (eh); + ExceptionHandlers.Remove (eh); } VariableDefinition? GetVariableReference (Instruction instruction) @@ -1356,16 +1368,16 @@ void CleanExceptionHandlers () switch (instruction.OpCode.Code) { case Code.Stloc_0: case Code.Ldloc_0: - return body.Variables[0]; + return Variables[0]; case Code.Stloc_1: case Code.Ldloc_1: - return body.Variables[1]; + return Variables[1]; case Code.Stloc_2: case Code.Ldloc_2: - return body.Variables[2]; + return Variables[2]; case Code.Stloc_3: case Code.Ldloc_3: - return body.Variables[3]; + return Variables[3]; } if (instruction.Operand is VariableReference vr) @@ -1410,7 +1422,9 @@ public bool Analyze (in CalleePayload callee, Stack callStack) { MethodDefinition method = callee.Method; Instruction[]? arguments = callee.Arguments; +#pragma warning disable RS0030 // This optimizer is the reason for the banned API, so it needs to use the Cecil directly Collection instructions = callee.Method.Body.Instructions; +#pragma warning restore RS0030 MethodBody body = method.Body; VariableReference vr; @@ -1844,8 +1858,12 @@ bool ConvertStackToResult () if (!body.InitLocals) return null; +#pragma warning disable RS0030 // This optimizer is the reason for the banned API, so it needs to use the Cecil directly + var variables = body.Variables; +#pragma warning restore RS0030 + // local variables don't need to be explicitly initialized - return CodeRewriterStep.CreateConstantResultInstruction (context, body.Variables[index].VariableType); + return CodeRewriterStep.CreateConstantResultInstruction (context, variables[index].VariableType); } bool GetOperandConstantValue ([NotNullWhen (true)] out object? value) diff --git a/src/linker/Linker/LinkContext.cs b/src/linker/Linker/LinkContext.cs index a10a7ac3e4f0..84855f7e252d 100644 --- a/src/linker/Linker/LinkContext.cs +++ b/src/linker/Linker/LinkContext.cs @@ -80,6 +80,7 @@ public class LinkContext : IMetadataResolver, ITryResolveMetadata, IDisposable readonly List _cachedWarningMessageContainers; readonly ILogger _logger; readonly Dictionary _isTrimmable; + readonly UnreachableBlocksOptimizer _unreachableBlocksOptimizer; public Pipeline Pipeline { get { return _pipeline; } @@ -225,6 +226,7 @@ public LinkContext (Pipeline pipeline, ILogger logger, string outputDirectory) GeneralSingleWarn = false; SingleWarn = new Dictionary (); AssembliesWithGeneratedSingleWarning = new HashSet (); + _unreachableBlocksOptimizer = new UnreachableBlocksOptimizer (this); const CodeOptimizations defaultOptimizations = CodeOptimizations.BeforeFieldInit | @@ -891,6 +893,28 @@ public int GetTargetRuntimeVersion () : null; } + readonly HashSet _processed_bodies_for_method = new HashSet (2048); + + /// + /// Linker applies some optimization on method bodies. For example it can remove dead branches of code + /// based on constant propagation. To avoid overmarking, all code which processes the method's IL + /// should only view the IL after it's been optimized. + /// As such typically MethodDefinition.MethodBody should not be accessed directly on the Cecil object model + /// instead all accesses to method body should go through the ILProvider here + /// which will make sure the IL of the method is fully optimized before it's handed out. + /// + public MethodIL GetMethodIL (Cecil.Cil.MethodBody methodBody) + => GetMethodIL (methodBody.Method); + + public MethodIL GetMethodIL (MethodDefinition method) + { + if (_processed_bodies_for_method.Add (method)) { + _unreachableBlocksOptimizer.ProcessMethod (method); + } + + return MethodIL.Create (method.Body); + } + readonly HashSet unresolved_reported = new (); protected virtual void ReportUnresolved (FieldReference fieldReference) diff --git a/src/linker/Linker/MethodBodyScanner.cs b/src/linker/Linker/MethodBodyScanner.cs index 318c0930b5a8..010178469e34 100644 --- a/src/linker/Linker/MethodBodyScanner.cs +++ b/src/linker/Linker/MethodBodyScanner.cs @@ -1,7 +1,6 @@ // Copyright (c) .NET Foundation and contributors. All rights reserved. // Licensed under the MIT license. See LICENSE file in the project root for full license information. -using System; using System.Collections.Generic; using System.Diagnostics.CodeAnalysis; using System.Linq; @@ -12,7 +11,7 @@ namespace Mono.Linker { public static class MethodBodyScanner { - public static bool IsWorthConvertingToThrow (MethodBody body) + public static bool IsWorthConvertingToThrow (MethodIL body) { // Some bodies are cheaper size wise to leave alone than to convert to a throw Instruction? previousMeaningful = null; @@ -64,9 +63,9 @@ public InterfacesOnStackScanner (LinkContext context) this.context = context; } - public IEnumerable<(InterfaceImplementation, TypeDefinition)>? GetReferencedInterfaces (MethodBody body) + public IEnumerable<(InterfaceImplementation, TypeDefinition)>? GetReferencedInterfaces (MethodIL methodIL) { - var possibleStackTypes = AllPossibleStackTypes (body.Method); + var possibleStackTypes = AllPossibleStackTypes (methodIL); if (possibleStackTypes.Count == 0) return null; @@ -95,27 +94,23 @@ public InterfacesOnStackScanner (LinkContext context) return interfaceImplementations; } - HashSet AllPossibleStackTypes (MethodDefinition method) + HashSet AllPossibleStackTypes (MethodIL methodIL) { - if (!method.HasBody) - throw new ArgumentException ("Method does not have body", nameof (method)); - - var body = method.Body; var types = new HashSet (); - foreach (VariableDefinition var in body.Variables) + foreach (VariableDefinition var in methodIL.Variables) AddIfResolved (types, var.VariableType); - foreach (var parameter in body.Method.Parameters) + foreach (var parameter in methodIL.Method.Parameters) AddIfResolved (types, parameter.ParameterType); - foreach (ExceptionHandler eh in body.ExceptionHandlers) { + foreach (ExceptionHandler eh in methodIL.ExceptionHandlers) { if (eh.HandlerType == ExceptionHandlerType.Catch) { AddIfResolved (types, eh.CatchType); } } - foreach (Instruction instruction in body.Instructions) { + foreach (Instruction instruction in methodIL.Instructions) { if (instruction.Operand is FieldReference fieldReference) { if (context.TryResolve (fieldReference)?.FieldType is TypeReference fieldType) AddIfResolved (types, fieldType); diff --git a/src/linker/Linker/MethodIL.cs b/src/linker/Linker/MethodIL.cs new file mode 100644 index 000000000000..fc97a6e24334 --- /dev/null +++ b/src/linker/Linker/MethodIL.cs @@ -0,0 +1,31 @@ +// Copyright (c) .NET Foundation and contributors. All rights reserved. +// Licensed under the MIT license. See LICENSE file in the project root for full license information. + +using Mono.Cecil; +using Mono.Cecil.Cil; +using Mono.Collections.Generic; + +namespace Mono.Linker +{ + /// + /// This is a wrapper which should be used by anything accessing method's body - see LinkContext.GetMethodIL for more details. + /// Any accesses made throught this wrapper are considered "safe"/OK since the wrapper is only created + /// once all of the optimizations are applied. + /// + public readonly record struct MethodIL + { + MethodIL (MethodBody body) => this.Body = body; + + public readonly MethodBody Body; + + public MethodDefinition Method => Body.Method; + +#pragma warning disable RS0030 // Wrapper which provides safe access to the property + public Collection Instructions => Body.Instructions; + public Collection ExceptionHandlers => Body.ExceptionHandlers; + public Collection Variables => Body.Variables; +#pragma warning restore RS0030 + + public static MethodIL Create (MethodBody body) => new MethodIL (body); + } +} diff --git a/src/linker/Linker/TypeReferenceWalker.cs b/src/linker/Linker/TypeReferenceWalker.cs index c4ad8e58d987..71ffbd1e521e 100644 --- a/src/linker/Linker/TypeReferenceWalker.cs +++ b/src/linker/Linker/TypeReferenceWalker.cs @@ -153,6 +153,7 @@ void WalkTypeScope (Collection forwarders) void WalkTypeScope (MethodBody body) { +#pragma warning disable RS0030 // Processing type references should not trigger method marking/processing, so access Cecil directly if (body.HasVariables) { foreach (var v in body.Variables) { WalkScopeOfTypeReference (v.VariableType); @@ -204,6 +205,7 @@ void WalkTypeScope (MethodBody body) } } } +#pragma warning restore RS0030 // Do not used banned APIs } void WalkMethodReference (MethodReference mr) diff --git a/test/Mono.Linker.Tests.Cases/DataFlow/CompilerGeneratedCodeInPreservedAssemblyWithWarning.cs b/test/Mono.Linker.Tests.Cases/DataFlow/CompilerGeneratedCodeInPreservedAssemblyWithWarning.cs index b66b0cd30929..f05f1b7fbb9c 100644 --- a/test/Mono.Linker.Tests.Cases/DataFlow/CompilerGeneratedCodeInPreservedAssemblyWithWarning.cs +++ b/test/Mono.Linker.Tests.Cases/DataFlow/CompilerGeneratedCodeInPreservedAssemblyWithWarning.cs @@ -26,12 +26,13 @@ public static void Main () WithLocalFunction (); } + // The compiler generated state will see the modified body, + // and will not associate the local function with the user method. + // Generic argument warnings from the local function will not be suppressed + // by RUC on the user method. + class Inner { - // In this case the compiler generated state will see the modified body, - // and will not associate the local function with the user method. - // Generic argument warnings from the local function will not be suppressed - // by RUC on the user method. [RequiresUnreferencedCode ("--" + nameof (Inner) + "." + nameof (WithLocalFunctionInner) + "--")] public static void WithLocalFunctionInner () { @@ -49,10 +50,6 @@ void LocalWithWarning () } } - // In this case the compiler generated state will see the original body, - // and will associate the local function with the user method. - // Generic argument warnings from the local function will be suppressed - // by RUC on the user method. [RequiresUnreferencedCode ("--" + nameof (WithLocalFunction) + "--")] public static void WithLocalFunction () { @@ -61,6 +58,7 @@ public static void WithLocalFunction () } // https://github.com/dotnet/linker/issues/2937 + [ExpectedWarning ("IL2091", ProducedBy = ProducedBy.Trimmer)] void LocalWithWarning () { // No warning diff --git a/test/Mono.Linker.Tests.Cases/UnreachableBlock/CompilerGeneratedCodeSubstitutions.cs b/test/Mono.Linker.Tests.Cases/UnreachableBlock/CompilerGeneratedCodeSubstitutions.cs new file mode 100644 index 000000000000..5b1edc37671a --- /dev/null +++ b/test/Mono.Linker.Tests.Cases/UnreachableBlock/CompilerGeneratedCodeSubstitutions.cs @@ -0,0 +1,245 @@ +using System; +using System.Collections.Generic; +using System.Diagnostics.CodeAnalysis; +using System.Threading.Tasks; +using Mono.Linker.Tests.Cases.Expectations.Assertions; +using Mono.Linker.Tests.Cases.Expectations.Metadata; + +namespace Mono.Linker.Tests.Cases.UnreachableBlock +{ + [SetupLinkerArgument ("--enable-opt", "ipconstprop")] + + // Using Kept validation on compiler generated code is tricky as we would have to describe + // all of the compiler generated classes and members which are expected to be kept. + // So not using that here (at least until we come up with a better way to do this). + // Instead this test relies on RUC and warnings to detect "kept" and "removed" calls. + [SkipKeptItemsValidation] + [ExpectedNoWarnings] + class CompilerGeneratedCodeSubstitutions + { + static void Main () + { + Lambda.Test (); + LocalFunction.Test (); + Iterator.Test (); + Async.Test (); + } + + class Lambda + { + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static void TestBranchInLambda () + { + var a = () => { + if (AlwaysFalse) { + RemovedMethod (); + } else { + UsedMethod (); + } + }; + + a (); + } + + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static void TestBranchAroundLambda () + { + Action a; + if (AlwaysFalse) { + a = () => RemovedMethod (); + } else { + a = () => UsedMethod (); + } + + a (); + } + + public static void Test () + { + TestBranchInLambda (); + TestBranchAroundLambda (); + } + } + + class LocalFunction + { + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static void TestBranchInLocalFunction () + { + void LocalFunction () + { + if (AlwaysFalse) { + RemovedMethod (); + } else { + UsedMethod (); + } + } + + LocalFunction (); + } + + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static void TestBranchAroundLocalFunction () + { + Action a; + if (AlwaysFalse) { + void RemovedLocalFunction () + { + RemovedMethod (); + } + + RemovedLocalFunction (); + } else { + void UsedLocalFunction () + { + UsedMethod (); + } + + UsedLocalFunction (); + } + } + + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static void TestBranchAroundUsageOfLocalFunction () + { + Action a; + if (AlwaysFalse) { + RemovedLocalFunction (); + } else { + UsedLocalFunction (); + } + + void RemovedLocalFunction () + { + RemovedMethod (); + } + + void UsedLocalFunction () + { + UsedMethod (); + } + } + + public static void Test () + { + TestBranchInLocalFunction (); + TestBranchAroundLocalFunction (); + TestBranchAroundUsageOfLocalFunction (); + } + } + + class Iterator + { + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static IEnumerable TestBranchWithNormalCall () + { + if (AlwaysFalse) { + RemovedMethod (); + } else { + UsedMethod (); + } + + yield return 1; + } + + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static IEnumerable TestBranchWithYieldAfter () + { + if (AlwaysFalse) { + RemovedMethod (); + yield return 1; + } else { + UsedMethod (); + yield return 1; + } + + yield return 1; + } + + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + // https://github.com/dotnet/linker/issues/3087 + [ExpectedWarning ("IL2026", "--RemovedMethod--", CompilerGeneratedCode = true)] + static IEnumerable TestBranchWithYieldBefore () + { + if (AlwaysFalse) { + yield return 1; + RemovedMethod (); + } else { + yield return 1; + UsedMethod (); + } + + yield return 1; + } + + public static void Test () + { + TestBranchWithNormalCall (); + TestBranchWithYieldAfter (); + TestBranchWithYieldBefore (); + } + } + + class Async + { + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + static async Task TestBranchWithNormalCall () + { + if (AlwaysFalse) { + RemovedMethod (); + } else { + UsedMethod (); + } + + await Task.FromResult (0); + } + + [ExpectedWarning ("IL2026", "--UsedMethod--", CompilerGeneratedCode = true)] + // https://github.com/dotnet/linker/issues/3087 + [ExpectedWarning ("IL2026", "--RemovedMethod--", CompilerGeneratedCode = true)] + static async Task TestBranchWithNormalCallAfterWAwait () + { + if (AlwaysFalse) { + await Task.FromResult (0); + RemovedMethod (); + } else { + await Task.FromResult (0); + UsedMethod (); + } + + await Task.FromResult (0); + } + + [ExpectedWarning ("IL2026", "--UsedAsyncMethod--", CompilerGeneratedCode = true)] + static async Task TestBranchWithAsyncCall () + { + if (AlwaysFalse) { + await RemovedAsyncMethod (); + } else { + await UsedAsyncMethod (); + } + } + + public static void Test () + { + TestBranchWithNormalCall ().RunSynchronously (); ; + TestBranchWithNormalCallAfterWAwait ().RunSynchronously (); + TestBranchWithAsyncCall ().RunSynchronously (); + } + } + + static bool AlwaysFalse => false; + + [RequiresUnreferencedCode ("--UsedAsyncMethod--")] + static async Task UsedAsyncMethod () => await Task.FromResult (0); + + [RequiresUnreferencedCode ("--RemovedAsyncMethod--")] + static async Task RemovedAsyncMethod () => await Task.FromResult (-1); + + [RequiresUnreferencedCode ("--UsedMethod--")] + static void UsedMethod () { } + + [RequiresUnreferencedCode ("--RemovedMethod--")] + static void RemovedMethod () { } + } +}