[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