[PATCH] D69374: [AArch64] Select sadd_sat, uadd_sat, usub_sat and ssub_sat.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 24 16:12:39 PDT 2019


efriedma added a comment.

Do we care about FPSR.QC?  I remember we had some discussion about it in the past, but I'm not remembering off the top of my head.

I agree it doesn't make sense to move an integer from a GPR to an FPR just to perform a saturating add.  Maybe GlobalISel could do something more clever here.

Otherwise looks fine.


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

https://reviews.llvm.org/D69374





More information about the llvm-commits mailing list