[PATCH] D109369: [AArch64][SVE] Add missing patterns for unpredicated subr intrinsics
Paul Walker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 7 09:46:59 PDT 2021
paulwalker-arm added a comment.
In D109369#2987120 <https://reviews.llvm.org/D109369#2987120>, @bsmith wrote:
> In D109369#2987058 <https://reviews.llvm.org/D109369#2987058>, @paulwalker-arm wrote:
>
>>
>
> That said, now that I think about it again, perhaps we could lower int_aarch64_sve_subr to a normal sub with reversed operands, which then I think would give us the isel for free, unless this is what you meant?
Yes, this is exactly what I meant. I can see that we already lower `int_aarch64_sve_sub` to `ISD::SUB` and we have `SUBR` patterns for `sub` so the missing link looks like just lowering `int_aarch64_sve_subr` to `ISD::SUB` as well.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109369/new/
https://reviews.llvm.org/D109369
More information about the llvm-commits
mailing list