[llvm-commits] [llvm] r60643 - /llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp

Chris Lattner sabre at nondot.org
Sat Dec 6 16:38:27 PST 2008


Author: lattner
Date: Sat Dec  6 18:38:27 2008
New Revision: 60643

URL: http://llvm.org/viewvc/llvm-project?rev=60643&view=rev
Log:
I love how using out of scope variables is not an error with GCC, no really I do.

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

Modified: llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp?rev=60643&r1=60642&r2=60643&view=diff

==============================================================================
--- llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp (original)
+++ llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp Sat Dec  6 18:38:27 2008
@@ -118,7 +118,7 @@
   Value *MemPtr = 0;
   uint64_t MemSize = 0;
   
-  if (StoreInst* S = dyn_cast<StoreInst>(QueryInst)) {
+  if (StoreInst *S = dyn_cast<StoreInst>(QueryInst)) {
     // If this is a volatile store, don't mess around with it.  Just return the
     // previous instruction as a clobber.
     if (S->isVolatile())
@@ -126,10 +126,10 @@
 
     MemPtr = S->getPointerOperand();
     MemSize = TD->getTypeStoreSize(S->getOperand(0)->getType());
-  } else if (LoadInst* LI = dyn_cast<LoadInst>(QueryInst)) {
+  } else if (LoadInst *LI = dyn_cast<LoadInst>(QueryInst)) {
     // If this is a volatile load, don't mess around with it.  Just return the
     // previous instruction as a clobber.
-    if (S->isVolatile())
+    if (LI->isVolatile())
       return MemDepResult::getClobber(--ScanIt);
     
     MemPtr = LI->getPointerOperand();





More information about the llvm-commits mailing list