silvas added a comment. makes sense. can you include a testcase? I assume we are forgetting some sort of diagnostic without this patch.
Also, if learning about Clang's modules, I would start with `addHeaderInclude` in lib/Frontend/FrontendActions.cpp. The module map stuff ultimately boils down to that (essentially, it textually forms a header which it then more-or-less compiles as a PCH; the submodule stuff is only relevant for setting Decl::Hidden and some extra checking when importing). http://reviews.llvm.org/D17787 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits