[PATCH] Header dependencies support for modularize

John Thompson john.thompson.jtsoftware at gmail.com
Tue Aug 13 11:10:34 PDT 2013


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/D1383

Files:
  test/modularize/NoProblemsDependencies.modularize
  test/modularize/Inputs/SomeOtherTypes.h
  test/modularize/Inputs/IsDependent.h
  modularize/Modularize.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1383.1.patch
Type: text/x-patch
Size: 10149 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130813/64c057fc/attachment.bin>


More information about the cfe-commits mailing list