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

Daniel Egger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 5 06:27:41 PDT 2021


therealprof added a comment.

In D105413#2858000 <https://reviews.llvm.org/D105413#2858000>, @dmgreen wrote:

> Hello. Looks like a good patch to me.



> This is very similar to the code in LowerSADDSUBSAT. Can we combine the functions into one?

The opcode selection would be vastly more complex but I can certainly take on a rework of that function if desirable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105413



More information about the llvm-commits mailing list