[PATCH] Consider module depedencies when checking a preamble in libclang

Ben Langmuir blangmuir at apple.com
Mon Jun 30 12:00:14 PDT 2014


This patch adds module dependencies (header files, module map files) to the list of files to check when deciding whether to rebuild a preamble.  That fixes using preambles with module imports so long as they are in non-overridden files. Overridden files require a change to ASTReader that I will put in a separate patch.

The fix is accomplished by adding yet another dependency collector to the CompilerInstance, but my intent is to use to unify the existing dependency collectors to the new “DependencyCollectory” interface, starting with the DependencyFileGenerator.  The default implementation of the DependencyCollector is intended to produce the same results as the dependency file generator.

Ben

-------------- next part --------------
A non-text attachment was scrubbed...
Name: reparsemodule.patch
Type: application/octet-stream
Size: 13087 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140630/ba67e010/attachment.obj>


More information about the cfe-commits mailing list