[patch 1/3] Header file reduction - backend files.
Andrew MacLeod
amacleod@redhat.com
Thu Oct 22 22:36:00 GMT 2015
On 10/22/2015 06:25 PM, Jeff Law wrote:
> On 10/01/2015 08:33 PM, Andrew MacLeod wrote:
>> these are all in the main gcc directory. 297 files total.
>>
>> Everything bootstraps on x86_64-pc-linux-gnu and
>> powerpc64le-unknown-linux-gnu. All targets in config-list.mk still
>> build. Regressions tests also came up clean.
>>
>> OK for trunk?
> So just to be clear, I'm looking at the backend-reorder patch. Now
> that I know it's going to be reordering & removing duplicates, I know
> better what to expect.
>
> Can you please look at data-streamer-{in,out}.c, data-streamer.c,
> gimplify-me.c. Maybe its changed recently, but options.h is removed
> as a duplicate by your patch, but on the trunk, I only see it showing
> up once (using your tools)
>
show-headers data-streamer.c -soptions.h
on trunk shows me:
alias.h
backend.h
tm.h
options.h (1) <<-------
flag-types.h
i386-opts.h
<...>
hard-reg-set.h
options.h (2) <<-------
fold-const.h
so the first occurrence is loaded via tm.h from backend.h
>
>
> I realize it's gotten a little stale, but IMHO once approved, if you
> need to make adjustments due to changes since you originally produced
> the patch those adjustments are pre-approved.
yeah. what I've been doing is apply the patch... and anything that fails
to apply, simply run the tool again on that file... its easier than
trying to fix things up :-)
Andrew
More information about the Gcc-patches
mailing list