Skip to content

Commit 5b14624

Browse files
committed
[X86][DAGCombine] Remove the predicate hasCCMP() in combineX86SubCmpForFlags, NFCI
No update for non-APX tests. This patch resolves TODO in #91747.
1 parent 9027ee6 commit 5b14624

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

llvm/lib/Target/X86/X86ISelLowering.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49239,8 +49239,7 @@ static SDValue combineX86SubCmpForFlags(SDNode *N, SDValue Flag,
4923949239

4924049240
SDValue SetCC = N->getOperand(0);
4924149241

49242-
// TODO: Remove the check hasCCMP() and update the non-APX tests.
49243-
if (!ST.hasCCMP() || SetCC.getOpcode() != X86ISD::SETCC || !Flag.hasOneUse())
49242+
if (SetCC.getOpcode() != X86ISD::SETCC || !Flag.hasOneUse())
4924449243
return SDValue();
4924549244

4924649245
// Check the only user of flag is `brcond ne`.

0 commit comments

Comments
 (0)