[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