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

Sat Jan 16 04:08:59 PST 2021

paulwalker-arm added a comment.

In D94708#2501255 <https://reviews.llvm.org/D94708#2501255>, @cameron.mcinally wrote:

> In D94444 <https://reviews.llvm.org/D94444>, @paulwalker-arm proposed a more generic extract vector intrinsic that accepts an index and stride. Now I'm wondering if we should just have a generic scalable shuffle vector intrinsic to handle all these operations under one intrinsic.
>
> That idea doesn't need to hold up this Diff, but it might be something to consider...

I don't believe that can work for the same reasons having a single LLVM instruction does not work.  Each shuffle has different requirements.  Some require one operand whilst other require two and some require additional scalar operands when others do not.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94708