PATCH: PR 42748

Mark Mitchell mark@codesourcery.com
Sun Feb 28 06:34:00 GMT 2010


Jason Merrill wrote:

>> This patch makes the mangler set the source location to the location
>> of the declaration while doing the mangling.
> 
> I've also been thinking about moving mangling messages to the decl locus
> instead, but by changing the warning call (as below) rather than by
> messing with input_location.

I don't think that's sufficiently general, at least without a bunch of
other changes.  In the case of the PR, the message comes from code in
the ARM back-end that has only a type, not a decl.  We'd have to find
some way to push the decl down.

>> I also made it push a
>> template instantiation level in the case that the manglee is a
>> template, so that we get the usual template instantiation backtrace in
>> that case.
> 
> Good idea.

So, do you think my patch is a good thing, or that we should pursue your
approach instead?

Thanks,

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713



More information about the Gcc-patches mailing list