[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
Tue Jul 6 09:43:16 PDT 2021


therealprof added a comment.

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

> 



> Yep, unifying them sounds better to me. It looks like its should be possible to keep the complexity of the opcode check down, and then it's simpler overall with a single function.

Done.


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