From 328a4fbfbb778de44d9a738c3efdfd42a58925d9 Mon Sep 17 00:00:00 2001 From: Jakub Staszak Date: Tue, 20 Aug 2013 23:04:15 +0000 Subject: [PATCH] Add some constantness. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@188844 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Analysis/BlockFrequencyImpl.h | 2 +- include/llvm/Analysis/CFG.h | 6 ++++-- lib/Analysis/CFG.cpp | 11 ++++++----- 3 files changed, 11 insertions(+), 8 deletions(-) diff --git a/include/llvm/Analysis/BlockFrequencyImpl.h b/include/llvm/Analysis/BlockFrequencyImpl.h index 5e14d6f09a5..817a44188b8 100644 --- a/include/llvm/Analysis/BlockFrequencyImpl.h +++ b/include/llvm/Analysis/BlockFrequencyImpl.h @@ -118,7 +118,7 @@ class BlockFrequencyImpl { /// isBackedge - Return if edge Src -> Dst is a reachable backedge. /// - bool isBackedge(BlockT *Src, BlockT *Dst) { + bool isBackedge(BlockT *Src, BlockT *Dst) const { unsigned a = RPO.lookup(Src); if (!a) return false; diff --git a/include/llvm/Analysis/CFG.h b/include/llvm/Analysis/CFG.h index 979926b18b3..e5683c8e595 100644 --- a/include/llvm/Analysis/CFG.h +++ b/include/llvm/Analysis/CFG.h @@ -65,7 +65,8 @@ bool isCriticalEdge(const TerminatorInst *TI, unsigned SuccNum, /// on branchy code but not loops, and LI is most useful on code with loops but /// does not help on branchy code outside loops. bool isPotentiallyReachable(const Instruction *From, const Instruction *To, - DominatorTree *DT = 0, LoopInfo *LI = 0); + const DominatorTree *DT = 0, + const LoopInfo *LI = 0); /// \brief Determine whether block 'To' is reachable from 'From', returning /// true if uncertain. @@ -74,7 +75,8 @@ bool isPotentiallyReachable(const Instruction *From, const Instruction *To, /// Returns false only if we can prove that once 'From' has been reached then /// 'To' can not be executed. Conservatively returns true. bool isPotentiallyReachable(const BasicBlock *From, const BasicBlock *To, - DominatorTree *DT = 0, LoopInfo *LI = 0); + const DominatorTree *DT = 0, + const LoopInfo *LI = 0); } // End llvm namespace diff --git a/lib/Analysis/CFG.cpp b/lib/Analysis/CFG.cpp index be7e9fce63c..c3f32d3a840 100644 --- a/lib/Analysis/CFG.cpp +++ b/lib/Analysis/CFG.cpp @@ -116,7 +116,7 @@ bool llvm::isCriticalEdge(const TerminatorInst *TI, unsigned SuccNum, // LoopInfo contains a mapping from basic block to the innermost loop. Find // the outermost loop in the loop nest that contains BB. -static const Loop *getOutermostLoop(LoopInfo *LI, const BasicBlock *BB) { +static const Loop *getOutermostLoop(const LoopInfo *LI, const BasicBlock *BB) { const Loop *L = LI->getLoopFor(BB); if (L) { while (const Loop *Parent = L->getParentLoop()) @@ -126,7 +126,7 @@ static const Loop *getOutermostLoop(LoopInfo *LI, const BasicBlock *BB) { } // True if there is a loop which contains both BB1 and BB2. -static bool loopContainsBoth(LoopInfo *LI, +static bool loopContainsBoth(const LoopInfo *LI, const BasicBlock *BB1, const BasicBlock *BB2) { const Loop *L1 = getOutermostLoop(LI, BB1); const Loop *L2 = getOutermostLoop(LI, BB2); @@ -135,7 +135,8 @@ static bool loopContainsBoth(LoopInfo *LI, static bool isPotentiallyReachableInner(SmallVectorImpl &Worklist, BasicBlock *StopBB, - DominatorTree *DT, LoopInfo *LI) { + const DominatorTree *DT, + const LoopInfo *LI) { // When the stop block is unreachable, it's dominated from everywhere, // regardless of whether there's a path between the two blocks. if (DT && !DT->isReachableFromEntry(StopBB)) @@ -179,7 +180,7 @@ static bool isPotentiallyReachableInner(SmallVectorImpl &Worklist, } bool llvm::isPotentiallyReachable(const BasicBlock *A, const BasicBlock *B, - DominatorTree *DT, LoopInfo *LI) { + const DominatorTree *DT, const LoopInfo *LI) { assert(A->getParent() == B->getParent() && "This analysis is function-local!"); @@ -191,7 +192,7 @@ bool llvm::isPotentiallyReachable(const BasicBlock *A, const BasicBlock *B, } bool llvm::isPotentiallyReachable(const Instruction *A, const Instruction *B, - DominatorTree *DT, LoopInfo *LI) { + const DominatorTree *DT, const LoopInfo *LI) { assert(A->getParent()->getParent() == B->getParent()->getParent() && "This analysis is function-local!"); -- 2.34.1