Fix typo & make the code leaner (dotnet/coreclr#25442)

Enum.HasFlag generates bigger IL and depends on complex JIT optimization for a good code. The classic bit check is strongly preferred accross CoreCLR and CoreFX. Signed-off-by: dotnet-bot <[email protected]>
csnewman · Jun 27, 2019 · bf0c913 · bf0c913
1 parent 08977c9
commit bf0c913
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/System.Private.CoreLib/shared/System/Diagnostics/StackTrace.cs b/src/System.Private.CoreLib/shared/System/Diagnostics/StackTrace.cs
@@ -340,11 +340,11 @@ private static bool ShowInStackTrace(MethodBase mb)
         {
             Debug.Assert(mb != null);
 
-            if (mb.MethodImplementationFlags.HasFlag(MethodImplAttributes.AggressiveInlining))
+            if ((mb.MethodImplementationFlags & MethodImplAttributes.AggressiveInlining) != 0)
             {
                 // Aggressive Inlines won't normally show in the StackTrace; however for Tier0 Jit and
                 // cross-assembly AoT/R2R these inlines will be blocked until Tier1 Jit re-Jits 
-                // them when they will inline. We don't show them in the StackTrace to bring consitency
+                // them when they will inline. We don't show them in the StackTrace to bring consistency
                 // between this first-pass asm and fully optimized asm.
                 return false;
             }