From: Davide Italiano <davide@freebsd.org>
Date: Sat, 25 Mar 2017 02:40:02 +0000 (+0000)
Subject: [NewGVN] Adjust NDEBUG markers.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b47fa05ce541fb67fb058362ee7975909c93a5b9;p=llvm

[NewGVN] Adjust NDEBUG markers.

This avoids 'used but not defined' warnings in Release builds
with GCC.

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

diff --git a/lib/Transforms/Scalar/NewGVN.cpp b/lib/Transforms/Scalar/NewGVN.cpp
index ab7400c562b..9dbafec09df 100644
--- a/lib/Transforms/Scalar/NewGVN.cpp
+++ b/lib/Transforms/Scalar/NewGVN.cpp
@@ -1928,6 +1928,7 @@ bool NewGVN::singleReachablePHIPath(const MemoryAccess *First,
 // subject to very rare false negatives. It is only useful for
 // testing/debugging.
 void NewGVN::verifyMemoryCongruency() const {
+#ifndef NDEBUG
   // Anything equivalent in the memory access table should be in the same
   // congruence class.
 
@@ -1982,6 +1983,7 @@ void NewGVN::verifyMemoryCongruency() const {
              "All MemoryPhi arguments should be in the same class");
     }
   }
+#endif
 }
 
 // Verify that the sparse propagation we did actually found the maximal fixpoint
@@ -2167,10 +2169,8 @@ bool NewGVN::runGVN() {
 
   initializeCongruenceClasses(F);
   iterateTouchedInstructions();
-#ifndef NDEBUG
   verifyMemoryCongruency();
   verifyIterationSettled(F);
-#endif
 
   Changed |= eliminateInstructions(F);