[llvm-commits] [llvm] r51407 - /llvm/trunk/lib/Analysis/AliasSetTracker.cpp

Chris Lattner sabre at nondot.org
Wed May 21 20:23:06 PDT 2008


Author: lattner
Date: Wed May 21 22:23:06 2008
New Revision: 51407

URL: http://llvm.org/viewvc/llvm-project?rev=51407&view=rev
Log:
Free and vaarg are not really volatile.

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

Modified: llvm/trunk/lib/Analysis/AliasSetTracker.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/AliasSetTracker.cpp?rev=51407&r1=51406&r2=51407&view=diff

==============================================================================
--- llvm/trunk/lib/Analysis/AliasSetTracker.cpp (original)
+++ llvm/trunk/lib/Analysis/AliasSetTracker.cpp Wed May 21 22:23:06 2008
@@ -284,20 +284,13 @@
 
 bool AliasSetTracker::add(FreeInst *FI) {
   bool NewPtr;
-  AliasSet &AS = addPointer(FI->getOperand(0), ~0,
-                            AliasSet::Mods, NewPtr);
-
-  // Free operations are volatile ops (cannot be moved).
-  AS.setVolatile();
+  addPointer(FI->getOperand(0), ~0, AliasSet::Mods, NewPtr);
   return NewPtr;
 }
 
 bool AliasSetTracker::add(VAArgInst *VAAI) {
   bool NewPtr;
-  AliasSet &AS = addPointer(VAAI->getOperand(0), ~0, AliasSet::ModRef, NewPtr);
-  
-  // Treat vaarg instructions as volatile (not to be moved).
-  AS.setVolatile();
+  addPointer(VAAI->getOperand(0), ~0, AliasSet::ModRef, NewPtr);
   return NewPtr;
 }
 





More information about the llvm-commits mailing list