[llvm] a282c68 - [X86] combineX86AddSub - pull out repeated getOperand() call. NFC.

[Simon Pilgrim via llvm-commits]

Author: Simon Pilgrim
Date: 2025-07-03T16:03:13+01:00
New Revision: a282c685806713321396c4230d32c0e2f79a1e8d

URL: https://github.com/llvm/llvm-project/commit/a282c685806713321396c4230d32c0e2f79a1e8d
DIFF: https://github.com/llvm/llvm-project/commit/a282c685806713321396c4230d32c0e2f79a1e8d.diff

LOG: [X86] combineX86AddSub - pull out repeated getOperand() call. NFC.

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86ISelLowering.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index f5d8d65070a3b..136d398359afd 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -57447,7 +57447,7 @@ static SDValue combineX86AddSub(SDNode *N, SelectionDAG &DAG,
   bool IsSub = X86ISD::SUB == N->getOpcode();
   unsigned GenericOpc = IsSub ? ISD::SUB : ISD::ADD;
 
-  if (IsSub && isOneConstant(N->getOperand(1)) && !N->hasAnyUseOfValue(0))
+  if (IsSub && isOneConstant(RHS) && !N->hasAnyUseOfValue(0))
     if (SDValue CMP = combineX86SubCmpForFlags(N, SDValue(N, 1), DAG, DCI, ST))
       return CMP;