[PATCH] D71636: [AArch64][SVE][WIP] Add support for vscale constants (?)

Chris Lattner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 18 22:12:34 PST 2019


lattner added a comment.

> My current thinking for shuffles is that we shouldn't represent the shuffle mask of shufflevector as a Constant at all. Currently, we basically treat it as an ArrayRef<int> anyway

I completely agree, this was a mistake in the representation and it would be really great to fix it!  I'd recommend moving the existing functionality before proposing extending shufflevector to vscale though.  One step at a time.  Thanks :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71636





More information about the llvm-commits mailing list