From 9cf812e1ed9c3db9c62d27289e947bfba8e635cb Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Sun, 18 Feb 2018 20:41:25 +0000 Subject: [PATCH] [X86] Correct a typo I made in combineToExtendCMOV recently. We're accidentally checking that the same node is a constant twice instead of checking the other node. This isn't a functional problem since we didn't do anything below that explicitly requires constants. It just means we may have introduced a sign_extend or zero_extend that won't fold out. llvm-svn: 325469 --- llvm/lib/Target/X86/X86ISelLowering.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index b3890b0409f5..df9c218d85d9 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -36066,7 +36066,7 @@ static SDValue combineToExtendCMOV(SDNode *Extend, SelectionDAG &DAG) { SDValue CMovOp1 = CMovN.getOperand(1); if (!isa(CMovOp0.getNode()) || - !isa(CMovOp0.getNode())) + !isa(CMovOp1.getNode())) return SDValue(); // Only extend to i32 or i64. -- 2.34.1