[PATCH] Header dependencies support for modularize

Thompson, John John_Thompson at playstation.sony.com
Fri Aug 16 05:42:11 PDT 2013


A quick review anyone?

Thanks.

-John

-----Original Message-----
From: cfe-commits-bounces at cs.uiuc.edu [mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of John Thompson
Sent: Tuesday, August 13, 2013 11:11 AM
To: john.thompson.jtsoftware at gmail.com
Cc: cfe-commits at cs.uiuc.edu
Subject: [PATCH] Header dependencies support for modularize

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




More information about the cfe-commits mailing list