[PATCH] D26267: [Modules] Include builtins with #include instead of #import for ObjC

Bruno Cardoso Lopes via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 4 12:31:15 PDT 2016


bruno marked an inline comment as done.
bruno added a comment.

In https://reviews.llvm.org/D26267#586971, @v.g.vassilev wrote:

> Could you include more context when creating the diff eg. git diff -U9999, or equivalent.


I did, -U99999 actually, not sure why you're not getting it...



================
Comment at: include/clang/Lex/ModuleMap.h:278
+  /// headers.
+  static bool isBuiltinHeader(StringRef FileName);
+
----------------
v.g.vassilev wrote:
> It seems this is in the private section and it is accessed by FrontendActions.cpp:224.
Oops!


https://reviews.llvm.org/D26267





More information about the cfe-commits mailing list