[PATCH] D26267: [Modules] Include builtins with #include instead of #import for ObjC
Bruno Cardoso Lopes via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 2 17:54:44 PDT 2016
bruno created this revision.
bruno added a reviewer: rsmith.
bruno added subscribers: cfe-commits, manmanren.
After r284797 builins are treated like textual includes. When compiling for ObjC++, the in-memory header file generated for the modules is composed only of #import's instead of includes. That could block some textual includes to happen because #import's will not re-enter headers already handled by other #import's.
Use #include when the header in question is a built-in.
https://reviews.llvm.org/D26267
Files:
include/clang/Lex/ModuleMap.h
lib/Frontend/FrontendActions.cpp
lib/Lex/ModuleMap.cpp
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/c++/v1/cstddef
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/c++/v1/math.h
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/c++/v1/module.modulemap
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/c++/v1/stddef.h
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/c++/v1/type_traits
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/module.modulemap
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/stddef.h
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/sys/_types/_ptrdiff_t.h
test/Modules/Inputs/libc-libcxx/sysroot/usr/include/sys/_types/_types.h
test/Modules/builtin-import.mm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26267.76807.patch
Type: text/x-patch
Size: 7978 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161103/0e0ea46b/attachment.bin>
More information about the cfe-commits
mailing list