This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Do not append " *INTERNAL* " to the decl name


ping...

Thanks,
Dehao

On Fri, Oct 18, 2013 at 11:06 AM, Dehao Chen <dehao@google.com> wrote:
> On Fri, Oct 18, 2013 at 10:39 AM, Jason Merrill <jason@redhat.com> wrote:
>> On 10/11/2013 01:59 PM, Dehao Chen wrote:
>>>
>>> It's hard to get a testcase without
>>> http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=201856 because
>>> none of these *INTERNAL* symbols will be emitted in debug info.
>>
>>
>> Why does that change cause one of these symbols to be emitted?  As Cary
>> says, that was done as an assertion.  If you're invalidating the assertion,
>> I need more explanation as to why.
>
> Sorry, pointed to the incorrect patch, it should be
> http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=201858
>
> These symbols will not be emitted in the symbol table in assembly, but
> they will be emitted in the debug_info after
> http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=201858, which
> emits function's linkage name in debug_info even when it's abstract.
> The *INTERNAL* assertion I removed is an over-assertion that not only
> assert for symbol table, but also for debug info.
>
> Thanks,
> Dehao
>
>>
>> Jason
>>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]