[PATCH] D141665: [DAGCombine][AArch64][ARM] Combine abd(sub(x, y)) to abd if the sub is nsw

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 16 08:21:50 PST 2023


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

LGTM - cheers



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:10120
+        TLI.isOperationLegalOrCustom(ISD::ABDS, VT))
+      return DAG.getNode(ISD::ABDS, SDLoc(N), VT, Op0, Op1);
     return SDValue();
----------------
You can probably hoist this before the sext/zext test?


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

https://reviews.llvm.org/D141665



More information about the llvm-commits mailing list