diff options
author | Peter Collingbourne <peter@pcc.me.uk> | 2016-05-26 01:19:17 +0000 |
---|---|---|
committer | Peter Collingbourne <peter@pcc.me.uk> | 2016-05-26 01:19:17 +0000 |
commit | ffecb1441b6f81b88971d3ddfcde67c7fc2b4032 (patch) | |
tree | 7d3f2b7c9a877496e02c76c0b811890fd2280042 | |
parent | 7c10dd1411e75c87f9de380642362244b144a8a4 (diff) | |
download | bcm5719-llvm-ffecb1441b6f81b88971d3ddfcde67c7fc2b4032.tar.gz bcm5719-llvm-ffecb1441b6f81b88971d3ddfcde67c7fc2b4032.zip |
MemorySSA: Remove argument to createNewAccess function.
There is only one caller of MemorySSA::createNewAccess, and it passes true
as the IgnoreNonMemory argument. Remove that argument and fold its behavior
into createNewAccess.
llvm-svn: 270812
-rw-r--r-- | llvm/include/llvm/Transforms/Utils/MemorySSA.h | 2 | ||||
-rw-r--r-- | llvm/lib/Transforms/Utils/MemorySSA.cpp | 7 |
2 files changed, 4 insertions, 5 deletions
diff --git a/llvm/include/llvm/Transforms/Utils/MemorySSA.h b/llvm/include/llvm/Transforms/Utils/MemorySSA.h index b6de7a98028..d2c494884fd 100644 --- a/llvm/include/llvm/Transforms/Utils/MemorySSA.h +++ b/llvm/include/llvm/Transforms/Utils/MemorySSA.h @@ -575,7 +575,7 @@ private: void computeDomLevels(DenseMap<DomTreeNode *, unsigned> &DomLevels); void markUnreachableAsLiveOnEntry(BasicBlock *BB); bool dominatesUse(const MemoryAccess *, const MemoryAccess *) const; - MemoryUseOrDef *createNewAccess(Instruction *, bool ignoreNonMemory = false); + MemoryUseOrDef *createNewAccess(Instruction *); MemoryAccess *findDominatingDef(BasicBlock *, enum InsertionPlace); void removeFromLookups(MemoryAccess *); diff --git a/llvm/lib/Transforms/Utils/MemorySSA.cpp b/llvm/lib/Transforms/Utils/MemorySSA.cpp index 896b24fc16e..b87a6d611b1 100644 --- a/llvm/lib/Transforms/Utils/MemorySSA.cpp +++ b/llvm/lib/Transforms/Utils/MemorySSA.cpp @@ -255,7 +255,7 @@ MemorySSAWalker *MemorySSA::buildMemorySSA(AliasAnalysis *AA, bool InsertIntoDef = false; AccessListType *Accesses = nullptr; for (Instruction &I : B) { - MemoryUseOrDef *MUD = createNewAccess(&I, true); + MemoryUseOrDef *MUD = createNewAccess(&I); if (!MUD) continue; InsertIntoDef |= isa<MemoryDef>(MUD); @@ -353,8 +353,7 @@ MemorySSAWalker *MemorySSA::buildMemorySSA(AliasAnalysis *AA, } /// \brief Helper function to create new memory accesses -MemoryUseOrDef *MemorySSA::createNewAccess(Instruction *I, - bool IgnoreNonMemory) { +MemoryUseOrDef *MemorySSA::createNewAccess(Instruction *I) { // Find out what affect this instruction has on memory. ModRefInfo ModRef = AA->getModRefInfo(I); bool Def = bool(ModRef & MRI_Mod); @@ -362,7 +361,7 @@ MemoryUseOrDef *MemorySSA::createNewAccess(Instruction *I, // It's possible for an instruction to not modify memory at all. During // construction, we ignore them. - if (IgnoreNonMemory && !Def && !Use) + if (!Def && !Use) return nullptr; assert((Def || Use) && |