rnk added a comment. One buglet, otherwise looks good.
================ Comment at: lib/Parse/ParsePragma.cpp:2169 @@ +2168,3 @@ + IdentifierInfo *II = Tok.getIdentifierInfo(); + if (II->getBuiltinID() < 2) + PP.Diag(Tok.getLocation(), diag::warn_pragma_intrinsic_builtin) ---------------- This doesn't seem like the right test. I think '1' is a valid builtin. This should probably be != 0. `__builtin_atan2` looks like it has builtin id 1, and you can use that to test. https://reviews.llvm.org/D23944 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits