OSDN Git Service

Fix redundant condition (PR32138)
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Fri, 10 Mar 2017 22:44:47 +0000 (22:44 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Fri, 10 Mar 2017 22:44:47 +0000 (22:44 +0000)
'!A || (A && B)' is equivalent to '!A || B'

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@297527 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/BranchCoalescing.cpp

index efc5d53..f2785be 100644 (file)
@@ -260,8 +260,8 @@ bool BranchCoalescing::canCoalesceBranch(CoalescingCandidateInfo &Cand) {
 
   // For now only consider triangles (i.e, BranchTargetBlock is set,
   // FalseMBB is null, and BranchTargetBlock is a successor to BranchBlock)
-  if (!Cand.BranchTargetBlock || (Cand.BranchTargetBlock && FalseMBB)
-      || !Cand.BranchBlock->isSuccessor(Cand.BranchTargetBlock)) { 
+  if (!Cand.BranchTargetBlock || FalseMBB ||
+      !Cand.BranchBlock->isSuccessor(Cand.BranchTargetBlock)) {
     DEBUG(dbgs() << "Does not form a triangle - skip\n");
     return false;
   }