This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [GOOGLE] use expand_location when checking same_line_p
- From: Dehao Chen <dehao at google dot com>
- To: Cary Coutant <ccoutant at google dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 1 Jul 2013 16:46:14 -0700
- Subject: Re: [GOOGLE] use expand_location when checking same_line_p
- References: <CAO2gOZWkp6S05ukHFhjO1CTh6svTrkraX5UO21ygB-sLn_Gheg at mail dot gmail dot com> <CAHACq4pOJzA68fxd3h-dmDJ_RwX5NWGhoLRrA0KsEu+4-GCH+Q at mail dot gmail dot com>
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 <ccoutant@google.com> wrote:
>> 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