[PATCH] D94708: [IR] Introduce llvm.experimental.vector.splice intrinsic

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 21 08:38:48 PST 2021


fhahn added a comment.

In D94708#2509677 <https://reviews.llvm.org/D94708#2509677>, @c-rhodes wrote:

> In D94708#2509610 <https://reviews.llvm.org/D94708#2509610>, @fhahn wrote:
>
>> Are there plans to use this for fixed vectors as well? If not, can this be restricted to scalable vectors only? It seems like for fixed vectors, preferring the `shufflevector` versions would be beneficial, to avoid having to update all transforms lookin at shuffles.
>>
>> Otherwise, this should probably have some tests for platforms other than AArch64 and also support in GlobalISel, which is the default on AArch64 with `-O0` IIRC (or do the transform to shuffles as an IR transform).
>
> The named shufflevector intrinsics accept both fixed and scalable vectors but for fixed they map to existing shufflevector.

As specified here yes, but I was wondering if they actually need to support fixed-width vectors? Is there a reason to use the intrinsics with fixed vectors instead of shuffles?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94708/new/

https://reviews.llvm.org/D94708



More information about the llvm-commits mailing list