[llvm] r290468 - Mark isOnlyReachableViaThisEdge as const

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 23 16:04:07 PST 2016


Author: dannyb
Date: Fri Dec 23 18:04:07 2016
New Revision: 290468

URL: http://llvm.org/viewvc/llvm-project?rev=290468&view=rev
Log:
Mark isOnlyReachableViaThisEdge as const

Modified:
    llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp?rev=290468&r1=290467&r2=290468&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp Fri Dec 23 18:04:07 2016
@@ -308,7 +308,7 @@ private:
   // Reachability handling.
   void updateReachableEdge(BasicBlock *, BasicBlock *);
   void processOutgoingEdges(TerminatorInst *, BasicBlock *);
-  bool isOnlyReachableViaThisEdge(const BasicBlockEdge &);
+  bool isOnlyReachableViaThisEdge(const BasicBlockEdge &) const;
   Value *findConditionEquivalence(Value *, BasicBlock *) const;
 
   // Elimination.
@@ -924,7 +924,7 @@ const Expression *NewGVN::performSymboli
 // There is an edge from 'Src' to 'Dst'.  Return true if every path from
 // the entry block to 'Dst' passes via this edge.  In particular 'Dst'
 // must not be reachable via another edge from 'Src'.
-bool NewGVN::isOnlyReachableViaThisEdge(const BasicBlockEdge &E) {
+bool NewGVN::isOnlyReachableViaThisEdge(const BasicBlockEdge &E) const {
 
   // While in theory it is interesting to consider the case in which Dst has
   // more than one predecessor, because Dst might be part of a loop which is




More information about the llvm-commits mailing list