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

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 11 05:56:22 PST 2019


SjoerdMeijer 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
----------------
Nit, just curious why this one is unaffected. Looking at the changes below, I was perhaps expecting similar changes here.


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

https://reviews.llvm.org/D69350





More information about the llvm-commits mailing list