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

Reply via email to