[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