[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 01:30:50 PDT 2021


therealprof added a comment.

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

>> The opcode selection would be vastly more complex but I can certainly take on a rework of that function if desirable.
>
> Yeah, it sounds like it should share more code than it complicates finding the new opcode.

Sorry I don't follow. Are you asking me to unify the functions or saying it's fine as is? 😅


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