diff options
author | George Burgess IV <george.burgess.iv@gmail.com> | 2016-08-03 19:57:02 +0000 |
---|---|---|
committer | George Burgess IV <george.burgess.iv@gmail.com> | 2016-08-03 19:57:02 +0000 |
commit | 024f3d2683e77428955a3151ba42ab53b031e4c0 (patch) | |
tree | 3c134c15d6583b21d02efa562fa8a3855e30007e /llvm/lib/Transforms/Utils | |
parent | 2e9675ff5247da937b1594f06fd5b8bbab7c5bf1 (diff) | |
download | bcm5719-llvm-024f3d2683e77428955a3151ba42ab53b031e4c0.tar.gz bcm5719-llvm-024f3d2683e77428955a3151ba42ab53b031e4c0.zip |
[MSSA] Add special handling for invariant/constant loads.
This is a follow-up to r277637. It teaches MemorySSA that invariant
loads (and loads of provably constant memory) are always liveOnEntry.
llvm-svn: 277640
Diffstat (limited to 'llvm/lib/Transforms/Utils')
-rw-r--r-- | llvm/lib/Transforms/Utils/MemorySSA.cpp | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Utils/MemorySSA.cpp b/llvm/lib/Transforms/Utils/MemorySSA.cpp index 16ece32a296..f90ab6eac2b 100644 --- a/llvm/lib/Transforms/Utils/MemorySSA.cpp +++ b/llvm/lib/Transforms/Utils/MemorySSA.cpp @@ -170,6 +170,7 @@ template <> struct DenseMapInfo<MemoryLocOrCall> { } }; } + namespace { struct UpwardsMemoryQuery { // True if our original query started off as a call @@ -251,6 +252,17 @@ static Reorderability getLoadReorderability(const LoadInst *Use, return Result; } +static bool isUseTriviallyOptimizableToLiveOnEntry(AliasAnalysis &AA, + const Instruction *I) { + // If the memory can't be changed, then loads of the memory can't be + // clobbered. + // + // FIXME: We should handle invariant groups, as well. It's a bit harder, + // because we need to pay close attention to invariant group barriers. + return isa<LoadInst>(I) && (I->getMetadata(LLVMContext::MD_invariant_load) || + AA.pointsToConstantMemory(I)); +} + static bool instructionClobbersQuery(MemoryDef *MD, const MemoryLocation &UseLoc, const Instruction *UseInst, @@ -1332,6 +1344,11 @@ void MemorySSA::OptimizeUses::optimizeUsesInBlock( continue; } + if (isUseTriviallyOptimizableToLiveOnEntry(*AA, MU->getMemoryInst())) { + MU->setDefiningAccess(MSSA->getLiveOnEntryDef()); + continue; + } + MemoryLocOrCall UseMLOC(MU); auto &LocInfo = LocStackInfo[UseMLOC]; // If the pop epoch changed, it means we've removed stuff from top of @@ -2246,6 +2263,12 @@ MemorySSA::CachingWalker::getClobberingMemoryAccess(MemoryAccess *MA) { if (auto *CacheResult = Cache.lookup(StartingAccess, Q.StartingLoc, Q.IsCall)) return CacheResult; + if (isUseTriviallyOptimizableToLiveOnEntry(*MSSA->AA, I)) { + MemoryAccess *LiveOnEntry = MSSA->getLiveOnEntryDef(); + Cache.insert(StartingAccess, LiveOnEntry, Q.StartingLoc, Q.IsCall); + return LiveOnEntry; + } + // Start with the thing we already think clobbers this location MemoryAccess *DefiningAccess = StartingAccess->getDefiningAccess(); |