[PATCH] D67052: Add reference type transformation builtins

Zoe Carver via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 3 18:05:12 PDT 2019


zoecarver added a comment.

In D67052#1656319 <https://reviews.llvm.org/D67052#1656319>, @mclow.lists wrote:

> If you're going to do `__add__lvalue_reference`, `__add_rvalue_reference`, and `__remove_reference`, why not go all the way and add `__is_reference`, `__is_lvalue_reference` and `__is_rvalue_reference`?


Those already exist :)

I am planning on adding others, `__remove_cv`, `__decay`, etc. but wanted to land this first and make sure I was doing it right.

I'll update `type_traits` in libc++ after this lands. I'm also planning on making a patch to update a few of the builtin type traits. That should significantly speed up build times. FYI here is all the "type trait primitives" clang supports. <https://clang.llvm.org/docs/LanguageExtensions.html#type-trait-primitives> At some point, we should try to support all of them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67052





More information about the cfe-commits mailing list