From: Craig Topper <craig.topper@gmail.com>
Date: Sun, 21 Jul 2013 18:58:40 +0000 (+0000)
Subject: Revert r186649 because it wasn't unnecessary and add a comment.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cbfb8d7ca40975acfa3382f5f62c40965fe316c4;p=clang

Revert r186649 because it wasn't unnecessary and add a comment.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@186795 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Basic/DiagnosticIDs.cpp b/lib/Basic/DiagnosticIDs.cpp
index e96d5097e6..898c51c497 100644
--- a/lib/Basic/DiagnosticIDs.cpp
+++ b/lib/Basic/DiagnosticIDs.cpp
@@ -92,13 +92,17 @@ static const unsigned StaticDiagInfoSize = llvm::array_lengthof(StaticDiagInfo);
 static const StaticDiagInfoRec *GetDiagInfo(unsigned DiagID) {
   // If assertions are enabled, verify that the StaticDiagInfo array is sorted.
 #ifndef NDEBUG
-  for (unsigned i = 1; i != StaticDiagInfoSize; ++i) {
-    assert(StaticDiagInfo[i-1].DiagID != StaticDiagInfo[i].DiagID &&
-           "Diag ID conflict, the enums at the start of clang::diag (in "
-           "DiagnosticIDs.h) probably need to be increased");
-
-    assert(StaticDiagInfo[i-1] < StaticDiagInfo[i] &&
-           "Improperly sorted diag info");
+  static bool IsFirst = true; // So the check is only performed on first call.
+  if (IsFirst) {
+    for (unsigned i = 1; i != StaticDiagInfoSize; ++i) {
+      assert(StaticDiagInfo[i-1].DiagID != StaticDiagInfo[i].DiagID &&
+             "Diag ID conflict, the enums at the start of clang::diag (in "
+             "DiagnosticIDs.h) probably need to be increased");
+
+      assert(StaticDiagInfo[i-1] < StaticDiagInfo[i] &&
+             "Improperly sorted diag info");
+    }
+    IsFirst = false;
   }
 #endif