rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land.
Please also document this trait in docs/LanguageExtensions.rst. ================ Comment at: include/clang/Basic/TypeTraits.h:91 +/// \brief Names for the array type traits. +enum ArrayTypeTrait { ATT_ArrayRank, ATT_ArrayExtent }; ---------------- Looks like you reformatted (and reindented) this whole file. If you meant to do that, please commit it as a separate change. ================ Comment at: lib/Parse/ParseDeclCXX.cpp:1407-1432 + if (TagType == DeclSpec::TST_struct && Tok.isNot(tok::identifier) && + !Tok.isAnnotation() && Tok.getIdentifierInfo() && + Tok.isOneOf( + tok::kw___is_abstract, tok::kw___is_aggregate, + tok::kw___is_arithmetic, tok::kw___is_array, tok::kw___is_assignable, + tok::kw___is_base_of, tok::kw___is_class, tok::kw___is_complete_type, + tok::kw___is_compound, tok::kw___is_const, tok::kw___is_constructible, ---------------- This list should only contain keywords that libstdc++ or libc++ has at one point defined as class templates; your trait should not be added here. ================ Comment at: lib/Parse/ParseExpr.cpp:903 REVERTIBLE_TYPE_TRAIT(__is_volatile); + REVERTIBLE_TYPE_TRAIT(__reference_binds_to_temporary); #undef REVERTIBLE_TYPE_TRAIT ---------------- Likewise. https://reviews.llvm.org/D29930 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits