Remove unnecessary IsReflectionBlocked check

GetTypesWithRuntimeMapping already filters out the reflection blocked types.
dotnet · Sep 24, 2018 · 67fcc10 · 67fcc10
1 parent ac12622
commit 67fcc10
Showing 1 changed file with 7 additions and 10 deletions.
diff --git a/src/ILCompiler.Compiler/src/Compiler/UsageBasedMetadataManager.cs b/src/ILCompiler.Compiler/src/Compiler/UsageBasedMetadataManager.cs
@@ -276,17 +276,14 @@ public MetadataManager ToAnalysisBasedMetadataManager()
 
             foreach (var constructedType in GetTypesWithRuntimeMapping())
             {
-                if (!IsReflectionBlocked(constructedType))
-                {
-                    reflectableTypes[constructedType] |= MetadataCategory.RuntimeMapping;
+                reflectableTypes[constructedType] |= MetadataCategory.RuntimeMapping;
 
-                    // Also set the description bit if the definition is getting metadata.
-                    TypeDesc constructedTypeDefinition = constructedType.GetTypeDefinition();
-                    if (constructedType != constructedTypeDefinition &&
-                        (reflectableTypes[constructedTypeDefinition] & MetadataCategory.Description) != 0)
-                    {
-                        reflectableTypes[constructedType] |= MetadataCategory.Description;
-                    }
+                // Also set the description bit if the definition is getting metadata.
+                TypeDesc constructedTypeDefinition = constructedType.GetTypeDefinition();
+                if (constructedType != constructedTypeDefinition &&
+                    (reflectableTypes[constructedTypeDefinition] & MetadataCategory.Description) != 0)
+                {
+                    reflectableTypes[constructedType] |= MetadataCategory.Description;
                 }
             }