[llvm-commits] CVS: llvm/lib/Analysis/AliasSetTracker.cpp

[Chris Lattner]

Changes in directory llvm/lib/Analysis:

AliasSetTracker.cpp updated: 1.50 -> 1.51
---
Log message:

when merging two alias sets together, be sure to propagate the volatility of
the inner set.  This fixes PR1435: http://llvm.org/PR1435  and Transforms/LICM/2007-05-22-VolatileSink.ll


---
Diffs of the changes:  (+5 -3)

 AliasSetTracker.cpp |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)


Index: llvm/lib/Analysis/AliasSetTracker.cpp
diff -u llvm/lib/Analysis/AliasSetTracker.cpp:1.50 llvm/lib/Analysis/AliasSetTracker.cpp:1.51
--- llvm/lib/Analysis/AliasSetTracker.cpp:1.50	Sun May  6 08:37:16 2007
+++ llvm/lib/Analysis/AliasSetTracker.cpp	Wed May 23 01:36:35 2007
@@ -351,9 +351,11 @@
       // Loop over all of the pointers in this alias set...
       AliasSet::iterator I = AS.begin(), E = AS.end();
       bool X;
-      for (; I != E; ++I)
-        addPointer(I.getPointer(), I.getSize(),
-                   (AliasSet::AccessType)AS.AccessTy, X);
+      for (; I != E; ++I) {
+        AliasSet &NewAS = addPointer(I.getPointer(), I.getSize(),
+                                     (AliasSet::AccessType)AS.AccessTy, X);
+        if (AS.isVolatile()) NewAS.setVolatile();
+      }
     }
 }