FYI, Phabricator doesn't send mail when you change the CC's on a review, so 
no-one was informed that this change existed until just now.

LGTM, thanks!

================
Comment at: test/Lexer/keywords_test.cpp:4-5
@@ +3,4 @@
+
+#define IS_KEYWORD(NAME) _Static_assert(!__is_identifier(NAME), #NAME)
+#define NOT_KEYWORD(NAME) _Static_assert(__is_identifier(NAME), #NAME)
+
----------------
It might be nice to also have an `IS_TYPE`, and check there that tentative 
parsing knows that the keyword is a type. Something like:

  #define IS_TYPE(NAME) void is_##TYPE##_type() { int(f(TYPE)); } // ok, 
function declaration

http://reviews.llvm.org/D3460



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to