On 07/08/2015 05:12 PM, Andrew MacLeod wrote:
While looking the initial results of an include reduction, there were a
couple of problematic files that are relatively easy to resolve.
This patch addresses flags.h. It includes both options.h and flag-types.h.
options.h already includes flag-types.h so including both is kind of
silly :-).
flags.h appears in 421 files... Only 25 of those files don't already
include one of options.h, backend.h, or tm.h (which also includes
options.h). Those are the only files which needed addressing.
This patch removes the 2 includes, and makes adjustments to the other 25
files.. either removing the include completely if not needed, replacing
it with options.h if possible, and as a last resort adding options.h to
the source file just before flags.h. In the end, only 2 actually
required flags.h itself.
Bootstraps on x86_64-unknown-linux-gnu, and regressions are currently
running. Also successfully built the 2 epiphany targets as there was a
config file touched for that target.
Assuming the regression run is clean, OK for trunk?
OK.
jeff
