From a917d4f9b4188fa330f9e89108e8c8d804a0ed22 Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Tue, 16 Aug 2011 01:28:22 +0000 Subject: Revert a bit of r137667; the logic in question can safely handle atomic load/store. llvm-svn: 137702 --- llvm/lib/Transforms/IPO/FunctionAttrs.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'llvm/lib/Transforms/IPO/FunctionAttrs.cpp') diff --git a/llvm/lib/Transforms/IPO/FunctionAttrs.cpp b/llvm/lib/Transforms/IPO/FunctionAttrs.cpp index c43f4b9870a..0edf3427507 100644 --- a/llvm/lib/Transforms/IPO/FunctionAttrs.cpp +++ b/llvm/lib/Transforms/IPO/FunctionAttrs.cpp @@ -163,15 +163,15 @@ bool FunctionAttrs::AddReadAttrs(const CallGraphSCC &SCC) { ReadsMemory = true; continue; } else if (LoadInst *LI = dyn_cast(I)) { - // Ignore non-volatile loads from local memory. - if (LI->isUnordered()) { + // Ignore non-volatile loads from local memory. (Atomic is okay here.) + if (!LI->isVolatile()) { AliasAnalysis::Location Loc = AA->getLocation(LI); if (AA->pointsToConstantMemory(Loc, /*OrLocal=*/true)) continue; } } else if (StoreInst *SI = dyn_cast(I)) { - // Ignore non-volatile stores to local memory. - if (SI->isUnordered()) { + // Ignore non-volatile stores to local memory. (Atomic is okay here.) + if (!SI->isVolatile()) { AliasAnalysis::Location Loc = AA->getLocation(SI); if (AA->pointsToConstantMemory(Loc, /*OrLocal=*/true)) continue; -- cgit v1.2.3