[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