fix what looks like a real logic bug, found by PVS-Studio (part of PR12357)
authorChris Lattner <sabre@nondot.org>
Tue, 27 Mar 2012 16:27:21 +0000 (16:27 +0000)
committerChris Lattner <sabre@nondot.org>
Tue, 27 Mar 2012 16:27:21 +0000 (16:27 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@153513 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/SelectionDAG/DAGCombiner.cpp

index 7c4db97bfead0a5b7bd09b916f1b38d336b06fb4..bac644a42a8ac25f0c0bbb521bf8327060ddfce6 100644 (file)
@@ -7991,8 +7991,8 @@ bool DAGCombiner::SimplifySelectOps(SDNode *TheSelect, SDValue LHS,
 
       if ((LLD->hasAnyUseOfValue(1) &&
            (LLD->isPredecessorOf(CondLHS) || LLD->isPredecessorOf(CondRHS))) ||
-          (LLD->hasAnyUseOfValue(1) &&
-           (LLD->isPredecessorOf(CondLHS) || LLD->isPredecessorOf(CondRHS))))
+          (RLD->hasAnyUseOfValue(1) &&
+           (RLD->isPredecessorOf(CondLHS) || RLD->isPredecessorOf(CondRHS))))
         return false;
 
       Addr = DAG.getNode(ISD::SELECT_CC, TheSelect->getDebugLoc(),