Re: [GOOGLE] use expand_location when checking same_line_p

2013-07-02 Thread Cary Coutant
>>> This patch replaces expand_location_to_spelling_point with >>> expand_location so that macros will not be expanded. OK for google/gcc-4_8. -cary

Re: [GOOGLE] use expand_location when checking same_line_p

2013-07-01 Thread Dehao Chen
This patch does not break the regression test. And because the final assembly is emitted with "expand_location" instead of "expand_location_to_spelling_point", thus when we check same_line_p, it should also reflect this. Dehao On Mon, Jul 1, 2013 at 4:28 PM, Cary Coutant wrote: >> This patch rep

Re: [GOOGLE] use expand_location when checking same_line_p

2013-07-01 Thread Cary Coutant
> This patch replaces expand_location_to_spelling_point with > expand_location so that macros will not be expanded. I'm confused. Doesn't this just undo the fix you made a month ago? How does this not break discriminator assignment? -cary

[GOOGLE] use expand_location when checking same_line_p

2013-07-01 Thread Dehao Chen
This patch replaces expand_location_to_spelling_point with expand_location so that macros will not be expanded. Bootstrapped and passed regression test. OK for google-4_8 branch? Thanks, Dehao Index: gcc/tree-cfg.c === --- gcc/tree