Re: [PATCH 1/2] (header usage fix) remove unused system header includes

I was hoping an RM would approve this as it seems just a hair beyond a normal darwin approval. I'm fine with this, and it does help darwin. Other ports should not care. On Sep 18, 2017, at 10:30 AM, Jack Howarth wrote: > > Pinging for the final gcc 5.5 release. >

Re: [PATCH 1/2] (header usage fix) remove unused system header includes

Pinging for the final gcc 5.5 release. On Mon, Aug 7, 2017 at 1:12 AM, Ryan Mounce wrote: > 2017-08-05 Ryan Mounce > > cherry picked from trunk r235361 > 2016-04-22 Szabolcs Nagy > > * auto-profile.c:

[PATCH 1/2] (header usage fix) remove unused system header includes

2017-08-05 Ryan Mounce cherry picked from trunk r235361 2016-04-22 Szabolcs Nagy * auto-profile.c: Remove include. * diagnostic.c: Remove include. * genmatch.c: Likewise. * pretty-print.c:

Re: [PATCH 1/2] (header usage fix) remove unused system header includes

On Fri, Apr 22, 2016 at 12:02 PM, Szabolcs Nagy wrote: > These headers include "system.h" which already includes > and so including them after "system.h" > is a noop and including them before may cause problems > if they depend on gcc macros from system.h. > >

[PATCH 1/2] (header usage fix) remove unused system header includes

These headers include "system.h" which already includes and so including them after "system.h" is a noop and including them before may cause problems if they depend on gcc macros from system.h. ipa-icf-gimple.c includes after system.h which poisons various libc symbols which may cause problems