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

Robin Kruppe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 10 12:55:04 PDT 2019


rkruppe added a comment.

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.


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

https://reviews.llvm.org/D32530





More information about the llvm-commits mailing list