[PATCH] D26267: [Modules] Include builtins with #include instead of #import for ObjC
Richard Smith via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 9 14:39:15 PST 2016
On 9 December 2016 at 03:59, Richard Smith via Phabricator via cfe-commits <
cfe-commits at lists.llvm.org> wrote:
> rsmith added a comment.
>
> Looks like we might only be making macros visible, and failing to emit the
> annot_module_import token for Sema.
Hmm, no, we inject that token immediately afterwards. I agree that the
module's contents should already be being made visible here. Can you look
at an AST dump and make sure that the declarations are owned by the correct
module? (You might need to try to use the declaration first, since we
populate the owning module information lazily.)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161209/78bb168a/attachment.html>
More information about the cfe-commits
mailing list