Move FailFast call so that we do not attempt to synchronize when the …

…condition is not met
dotnet · Aug 27, 2021 · 9a1e743 · 9a1e743
1 parent 4b9dcc5
commit 9a1e743
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/src/Compilers/Core/Portable/InternalUtilities/FailFast.cs b/src/Compilers/Core/Portable/InternalUtilities/FailFast.cs
@@ -91,7 +91,6 @@ internal static void DumpStackTrace(Exception? exception = null, string? message
         /// <param name="message">An optional message to be recorded in the dump in case of failure. Can be <c>null</c>.</param>
         [Conditional("DEBUG")]
         [DebuggerHidden]
-        [MethodImpl(MethodImplOptions.Synchronized)]
         internal static void Assert([DoesNotReturnIf(false)] bool condition, string? message = null)
         {
             if (condition)
@@ -104,8 +103,7 @@ internal static void Assert([DoesNotReturnIf(false)] bool condition, string? mes
                 Debugger.Break();
             }
 
-            DumpStackTrace(message: message);
-            Environment.FailFast("ASSERT FAILED" + Environment.NewLine + message);
+            Fail("ASSERT FAILED" + Environment.NewLine + message)
         }
     }
 }