[PATCH] D105413: [ARM] Add lowering of uadd_sat to uq{add|sub}8 and uq{add|sub}16

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Hello. Looks like a good patch to me.



================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:4988
 
+static SDValue LowerUADDSUBSAT(SDValue Op, SelectionDAG &DAG,
+                               const ARMSubtarget *Subtarget) {
----------------
This is very similar to the code in LowerSADDSUBSAT. Can we combine the functions into one?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105413