[PATCH] D65753: Builtins: Add some v2f16 variants
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 5 12:47:26 PDT 2019
rjmccall added a comment.
Hmm. I think there are two reasonable concerns here, both arising from the fact that these names occupy (the builtin namespace parallel to) a namespace that the committee might want to occupy in the future:
- Is a `v2fN` suffix sufficiently likely to avoid interfering with this namespace? If there's an *exact* collision, that might be okay if the builtin actually matches the standard behavior; my concern is that they might conflict. (Different orderings of element count and element type? Unrelated functions ending in `v`?)
- Is it a good idea to introduce builtins in this namespace that traffic in our current vector types instead of a potential future standard-blessed vector type?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65753/new/
https://reviews.llvm.org/D65753
More information about the cfe-commits
mailing list