[llvm] r259594 - [LVI] Fix debug output

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 2 14:43:08 PST 2016


Author: reames
Date: Tue Feb  2 16:43:08 2016
New Revision: 259594

URL: http://llvm.org/viewvc/llvm-project?rev=259594&view=rev
Log:
[LVI] Fix debug output

Due to staleness in a patch I committed yesterday, the debug output was reporting overdefined cases as being undefined.  Confusing to say the least.  The mistake appears to have only effected the debug output thankfully.


Modified:
    llvm/trunk/lib/Analysis/LazyValueInfo.cpp

Modified: llvm/trunk/lib/Analysis/LazyValueInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/LazyValueInfo.cpp?rev=259594&r1=259593&r2=259594&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/LazyValueInfo.cpp (original)
+++ llvm/trunk/lib/Analysis/LazyValueInfo.cpp Tue Feb  2 16:43:08 2016
@@ -576,7 +576,7 @@ void LazyValueInfoCache::solve() {
              "Result should be in cache!");
 
       DEBUG(dbgs() << "POP " << *e.second << " in " << e.first->getName()
-                   << " = " << lookup(e.second)[e.first] << "\n");
+                   << " = " << getCachedValueInfo(e.second, e.first) << "\n");
 
       BlockValueStack.pop();
       BlockValueSet.erase(e);
@@ -790,7 +790,7 @@ bool LazyValueInfoCache::solveBlockValue
     // to overdefined.
     if (Result.isOverdefined()) {
       DEBUG(dbgs() << " compute BB '" << BB->getName()
-            << "' - overdefined because of pred.\n");
+            << "' - overdefined because of pred (non local).\n");
       // If we previously determined that this is a pointer that can't be null
       // then return that rather than giving up entirely.
       if (NotNull) {
@@ -835,7 +835,7 @@ bool LazyValueInfoCache::solveBlockValue
     // to overdefined.
     if (Result.isOverdefined()) {
       DEBUG(dbgs() << " compute BB '" << BB->getName()
-            << "' - overdefined because of pred.\n");
+            << "' - overdefined because of pred (local).\n");
 
       BBLV = Result;
       return true;




More information about the llvm-commits mailing list