[PATCH] modularize - header dependencies - version 2
John Thompson
john.thompson.jtsoftware at gmail.com
Thu Aug 22 07:51:36 PDT 2013
Hi silvas, klimek,
Here's another crack at it using the options subsystems this time. (Thanks for the suggestions Sean and Manuel!)
The original summary:
In using modularize to check a large group of platform headers for modules-readiness, I found that a few headers had dependencies, such that they required other headers to be included first to avoid compile errors on missing definitions.
This patch adds support to modularize to allow specifying depended-on headers in the header file list input to modularize, i.e.
header.h: dependency1.h dependency2.h
http://llvm-reviews.chandlerc.com/D1474
Files:
test/modularize/NoProblemsDependencies.modularize
test/modularize/Inputs/SomeOtherTypes.h
test/modularize/Inputs/IsDependent.h
modularize/Modularize.cpp
modularize/CMakeLists.txt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1474.1.patch
Type: text/x-patch
Size: 10056 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130822/ea6a00c6/attachment.bin>
More information about the cfe-commits
mailing list