[PATCH] D91362: [SelectionDAG] Add llvm.vector.{extract,insert} intrinsics

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 12 09:25:58 PST 2020


lebedev.ri added a comment.

In D91362#2391668 <https://reviews.llvm.org/D91362#2391668>, @paulwalker-arm wrote:

> @lebedev.ri: shufflevector only has minimal support for scalable vectors with only the splat case covered (and even that has its quirks).  With the recent change to force the mask to be an ArrayRef there is no way to represent arbitrary shuffles and at the same time the implementation forced a requirement that scalable vector data inputs imply a scalable vector result.

I still want to see all this explanation be put both into an RFC and into patch's description.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91362



More information about the llvm-commits mailing list