[llvm] r350895 - [MemorySSA] Disable checkClobberSanity for SkipSelfWalker.

Alina Sbirlea via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 10 13:47:15 PST 2019


Author: asbirlea
Date: Thu Jan 10 13:47:15 2019
New Revision: 350895

URL: http://llvm.org/viewvc/llvm-project?rev=350895&view=rev
Log:
[MemorySSA] Disable checkClobberSanity for SkipSelfWalker.

Sanity will fail for this, since we're exploring getting a clobber
further than the sanity check expects.
Ideally we need to teach the sanity check to differentiate between the
two walkers based on the SkipSelf bool in the query.

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

Modified: llvm/trunk/lib/Analysis/MemorySSA.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/MemorySSA.cpp?rev=350895&r1=350894&r2=350895&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/MemorySSA.cpp (original)
+++ llvm/trunk/lib/Analysis/MemorySSA.cpp Thu Jan 10 13:47:15 2019
@@ -918,7 +918,8 @@ public:
     }
 
 #ifdef EXPENSIVE_CHECKS
-    checkClobberSanity(Current, Result, Q.StartingLoc, MSSA, Q, AA);
+    if (!Q.SkipSelfAccess)
+      checkClobberSanity(Current, Result, Q.StartingLoc, MSSA, Q, AA);
 #endif
     return Result;
   }




More information about the llvm-commits mailing list