[PATCH] D57281: [X86] combineCarryThroughADD - add support for X86::COND_A commutations (PR24545)

Sat Jan 26 12:28:52 PST 2019

craig.topper added inline comments.

================
Comment at: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp:34904
+          //
+          if (CarryOp1.getOpcode() == X86ISD::SUB && CarryOp1.hasOneUse() &&
+              CarryOp1.getValueType().isInteger() &&
----------------
Do we need to make sure the data output of the subtract doesn't have any users? CarryOp1.hasOneUse() will only check the flag result has one use. CarryOp1->hasOneUse() will check that the entire node only has one use.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57281/new/

https://reviews.llvm.org/D57281