rjmccall added a comment. Yes, this is very close, thank you.
================ Comment at: clang/include/clang/Basic/TokenKinds.def:805 +INTERESTING_IDENTIFIER(float_t) +INTERESTING_IDENTIFIER(double_t) +INTERESTING_IDENTIFIER(FILE) ---------------- I think it would be cleaner if you added these two as interesting identifiers in your follow-up patch. ================ Comment at: clang/lib/Sema/SemaLookup.cpp:950 + return true; + } // In C++ and OpenCL (spec v1.2 s6.9.f), we don't have any predefined ---------------- The two changes in this file shouldn't be necessary, since interesting identifiers should be disjoint from builtins. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146148/new/ https://reviews.llvm.org/D146148 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits