[PATCH 2/2] (header usage fix) include c++ headers in system.h

2017-08-06 Thread Ryan Mounce
2017-08-05 Ryan Mounce cherry picked from trunk r235362 2016-04-22 Szabolcs Nagy * system.h (list, map, set, vector): Include conditionally. * auto-profile.c (INCLUDE_MAP, INCLUDE_SET): Define. *

Re: [PATCH 2/2] (header usage fix) include c++ headers in system.h

2016-04-22 Thread Pedro Alves
On 04/22/2016 11:02 AM, Szabolcs Nagy wrote: > Some gcc source files include standard headers after > "system.h" but those headers may declare and use poisoned > symbols, Couldn't gcc simply allow use of poisoned symbols in system headers? It sounds like it'd avoid these odd contortions. >

Re: [PATCH 2/2] (header usage fix) include c++ headers in system.h

2016-04-22 Thread David Edelsohn
On Fri, Apr 22, 2016 at 6:02 AM, Szabolcs Nagy wrote: > Some gcc source files include standard headers after > "system.h" but those headers may declare and use poisoned > symbols, they also cannot be included before "system.h" > because they might depend on macro

Re: [PATCH 2/2] (header usage fix) include c++ headers in system.h

2016-04-22 Thread Richard Biener
On Fri, Apr 22, 2016 at 12:02 PM, Szabolcs Nagy wrote: > Some gcc source files include standard headers after > "system.h" but those headers may declare and use poisoned > symbols, they also cannot be included before "system.h" > because they might depend on macro

Re: [PATCH 2/2] (header usage fix) include c++ headers in system.h

2016-04-22 Thread James Greenhalgh
On Fri, Apr 22, 2016 at 11:02:48AM +0100, Szabolcs Nagy wrote: > Some gcc source files include standard headers after > "system.h" but those headers may declare and use poisoned > symbols, they also cannot be included before "system.h" > because they might depend on macro definitions from there, >

[PATCH 2/2] (header usage fix) include c++ headers in system.h

2016-04-22 Thread Szabolcs Nagy
Some gcc source files include standard headers after "system.h" but those headers may declare and use poisoned symbols, they also cannot be included before "system.h" because they might depend on macro definitions from there, so they must be included in system.h. This patch fixes the use of , , ,