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(); + } } } _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits