Re: [PATCH 01/11] Fix cpp_sys_macro_p with -ftrack-macro-expansion

2012-04-14 Thread Jason Merrill
OK. Jason

Re: [PATCH 01/11] Fix cpp_sys_macro_p with -ftrack-macro-expansion

2012-04-14 Thread Dodji Seketeli
Jason Merrill writes: > On 04/10/2012 10:55 AM, Dodji Seketeli wrote: >> + if (CPP_OPTION (pfile, track_macro_expansion)) > > I think this should check context->tokens_kind rather than the > compiler flag. OK. Below is the updated patch that does that. Tested and bootstrapped on x86_64-unknow

Re: [PATCH 01/11] Fix cpp_sys_macro_p with -ftrack-macro-expansion

2012-04-11 Thread Jason Merrill
On 04/10/2012 10:55 AM, Dodji Seketeli wrote: + if (CPP_OPTION (pfile, track_macro_expansion)) I think this should check context->tokens_kind rather than the compiler flag. Jason

[PATCH 01/11] Fix cpp_sys_macro_p with -ftrack-macro-expansion

2012-04-10 Thread Dodji Seketeli
Hello, cpp_sys_macro_p crashes when -ftrack-macro-expansion is on. The issue can be reproduced by running the tests: runtest --tool gcc --tool_opts="-ftrack-macro-expansion" cpp.exp=sysmac1.c runtest --tool gcc --tool_opts="-ftrack-macro-expansion" cpp.exp=sysmac2.c This is because it j