[PATCH] D32756: [DAGCombine] Refactor common addcarry pattern.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

@filcab suggested adding the PR number on the commit message title as well.

LGTM - couple of minors.



================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:2224
+    if (Y.getOpcode() == ISD::UADDO &&
+        CarryIn.getOpcode() == ISD::ADDCARRY &&
+        isNullConstant(CarryIn.getOperand(1)) &&
----------------
Worth checking/asserting that CarryIn.getResNo() == 1 ?


https://reviews.llvm.org/D32756