On Wed, Apr 11, 2012 at 3:59 AM, Dodji Seketeli <do...@redhat.com> wrote:
> Now that diagnostics first point to the spelling location of tokens
> coming from macro expansion, the test case
> gcc/testsuite/g++.old-deja/g++.other/vaarg3.C shows that when I write
> va_args (args, some_type), the location that is recorded for
> "some_type" is not correct. We wrongly record a location that is in
> the system header where the va_args macro is defined.
> This patch changes that to correctly record the location for the type
> operand of the va_arg expression.
> With this patch applied, the
> gcc/testsuite/g++.old-deja/g++.other/vaarg3.C test PASSes with and
> without -ftrack-macro-expansion.
> Tested on x86_64-unknown-linux-gnu against trunk.
> Note that the bootstrap with -ftrack-macro-expansion exhibits other
> separate issues that are addressed in subsequent patches. This patch
> just fixes one class of problems.
> The patch does pass bootstrap with -ftrack-macro-expansion turned off,