Annnnnd I finally got around to updating this patch! The new patch is simplified to rely on the ASTWriter’s concept of what a ‘system’ dependency is. To that end, I’ve changed module.map files to no longer be unconditionally ‘user’ files. Previously we forced module.map to be ‘user’ input files as a cheap way to detect system header changes without ‘stat’ing all the headers. With Dmitri’s work on a timestamp-file-based approach, we shouldn’t need that trick anymore.
Ben
print-module-deps.patch
Description: Binary data
On Jan 22, 2014, at 4:40 PM, Ben Langmuir <[email protected]> wrote: > This patch adds module dependencies to the dependency files created by > -MD/-MMD/etc. It does so by attaching an ASTReaderListener that will call > into the dependency file generator when a module input file is seen in the > serialized AST. In order to add/not add system headers appropriately, the > IsSystem flag from the clang::Module is passed through the ASTReader and > ModuleFile. There was also an addition to allow chaining ASTReaderListeners > (let me know if you’d prefer that to be a separate patch). > > Ben > > <module-deps.patch>_______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
