[PATCH] D32530: [SVE][IR] Scalable Vector IR Type

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 13 11:48:14 PDT 2019


greened added a comment.

In D32530#1498469 <https://reviews.llvm.org/D32530#1498469>, @rkruppe wrote:

> I know very well how annoying it can be to read and write (and say) the `scalable` prefix all the time and wish for something shorter sometimes, but I also prefer `<vscale x ...>` for the reasons Sander gave. I'll add that `<vscale x 4 x i32>` feels a bit lighter than `<scalable 4 x i32>` even though it's the same number of characters (maybe because there's more whitespace?).
>
> The `<n x ...>` syntax is shorter but doesn't have the mnemonic aspect and also clashes with the pre-existing use of "n" as a metavariable standing for some fixed vector length (as in, `<N x i1>` for example), so I'd rather have `<scalable ...>` if those are the two options.


+1


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

https://reviews.llvm.org/D32530





More information about the llvm-commits mailing list