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

Bruno Cardoso Lopes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 10 19:00:32 PST 2017


bruno closed this revision.
bruno added a comment.

Thanks for the review Richard.

Committed 291644



================
Comment at: lib/Lex/HeaderSearch.cpp:1082
 
+  auto TryEnterImported = [&](void) -> bool {
+    if (!ModulesEnabled)
----------------
rsmith wrote:
> Maybe add a FIXME here indicating that this is a workaround for the lack of proper modules-aware support for `#import` / `#pragma once`?
Done. 


https://reviews.llvm.org/D26267





More information about the cfe-commits mailing list