[PATCH] D69350: [ARM] Replace arm_neon_vqadds with sadd_sat

[Dave Green via Phabricator via llvm-commits]

dmgreen marked an inline comment as done.
dmgreen added inline comments.


================
Comment at: llvm/test/CodeGen/ARM/addsubo-legalization.ll:88
   %y = load <2 x i64>, <2 x i64>* %ptr2, align 8
   %s = call {<2 x i64>, <2 x i1>} @llvm.usub.with.overflow.v2i64(<2 x i64> %x, <2 x i64> %y)
   %m = extractvalue {<2 x i64>, <2 x i1>} %s, 0
----------------
SjoerdMeijer wrote:
> Nit, just curious why this one is unaffected. Looking at the changes below, I was perhaps expecting similar changes here.
Good question. It looks like the default expansion for SADD/SUBO can go via a SADD/SUBSAT, but the UADD/SUBO prefers to go via ADD/SUBCARRY.

We could add expansion for UADDO to UADDSAT, but I think it's best not to do that here.


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

https://reviews.llvm.org/D69350