[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