[PATCH] D143371: [DAG][AArch64][ARM] Recognize hadd from wrapping flags.

Mon Feb 6 09:00:13 PST 2023

dmgreen added a comment.

Thanks

================
Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:959-965
     if ((ConstOp = isConstOrConstSplat(Op1, DemandedElts)) &&
         ConstOp->isOne()) {
       ExtOpA = Op2;
       ExtOpB = Op3;
+      Add2 = A;
       return true;
     }
----------------
spatel wrote:
> It's semi-independent of this patch, but do we need this clause? We're looking for a "1" constant, so it would always get canonicalized to operand 1 of an incoming 'add'?
Good point. Will do.

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

https://reviews.llvm.org/D143371