[PATCH] D58965: [DAGCombiner] Enable UADDO/USUBO vector combine support

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 5 07:26:08 PST 2019


nikic added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:2463
   // If Y + 1 cannot overflow.
   if (N1.getOpcode() == ISD::ADDCARRY && isNullConstant(N1.getOperand(1))) {
     SDValue Y = N1.getOperand(0);
----------------
Should this also become isNullorNullSplat? (Does ADDCARRY support vectors?)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58965





More information about the llvm-commits mailing list