[llvm] r176120 - Debug Info: for static member variables, move AT_MIPS_linkage_name from

[Manman Ren]

Hi Paul and Eric,

I checked in an updated commit at r176143.
This commit adds AT_MIPS_linkage_name to the definition DIE.

Thanks,
Manman

On Feb 26, 2013, at 2:50 PM, "Robinson, Paul" <Paul.Robinson at am.sony.com> wrote:

> On Tue, Feb 26, 2013 at 12:48 PM, Manman Ren <mren at apple.com> wrote:
>> Debug Info: for static member variables, move AT_MIPS_linkage_name from
>> TAG_member inside a class to the specification DIE.
> 
> The declaration would be the "specification DIE" (i.e., the DIE referred
> to by the DW_AT_specification attribute) and that's where it was.
> You moved it to the definition DIE.
> 
>> Having AT_MIPS_linkage_name on TAG_member caused old gdb (GNU 6.3.50) to
>> error out. 
> 
> It is more likely that old GDB errored out because Clang now uses DW_TAG_member
> to declare static members inside a class, instead of DW_TAG_variable.
> 
>> Also gcc 4.7 has AT_MIPS_linkage_name on the specification DIE.
> 
> GCC 4.4.3 has it on the declaration DIE.
> 
> --paulr
>