Neaten up this method. Check that if there is only one

predecessor then it's Src.

llvm-svn: 149843

1 files changed, 3 insertions, 3 deletions

diff --git a/llvm/lib/Transforms/Scalar/GVN.cpp b/llvm/lib/Transforms/Scalar/GVN.cpp
index f6d6b90b99a..654125342a4 100644
--- a/llvm/lib/Transforms/Scalar/GVN.cpp
+++ b/llvm/lib/Transforms/Scalar/GVN.cpp
@@ -1999,10 +1999,10 @@ static bool isOnlyReachableViaThisEdge(BasicBlock *Src, BasicBlock *Dst,
   // only reachable from Src, in practice it is pointless since at the time
   // GVN runs all such loops have preheaders, which means that Dst will have
   // been changed to have only one predecessor, namely Src.
-  pred_iterator PI = pred_begin(Dst), PE = pred_end(Dst);
-  assert(PI != PE && "No edge between these basic blocks!");
+  BasicBlock *Pred = Dst->getSinglePredecessor();
+  assert((!Pred || Pred == Src) && "No edge between these basic blocks!");
   (void)Src;
-  return PE == ++PI;
+  return Pred != 0;
 }
 
 /// processInstruction - When calculating availability, handle an instruction