[PATCH] D18706: [DWARF] Force a linkage_name on an inlined subprogram's abstract origin

[Paul Robinson via llvm-commits]

probinson added a comment.

In http://reviews.llvm.org/D18706#391801, @probinson wrote:

> Built Clang with Clang 3 times: linkage names Enabled, linkage names Disabled but with the patch, linkage names Disabled without the patch (aka: all names, some names, no names).
>  'all names' and 'no names' are the options we have today; this patch replaces 'no names' with 'some names.'


dblaikie wrote (in email):

> Which option(s) are you referring to here ^ (do we have a flag or somesuch that 

>  allows enabling/disabling linkage names?)


Yes, DwarfDebug.cpp has `-dwarf-linkage-names={Enable,Disable}` which is disabled by default when tuning for SCE, otherwise enabled by default.  This is what `DD->useLinkageNames()` is referring to, in addLinkageName(), giving us the all/none options.
My patch obviously substitutes 'some' for 'none'.


http://reviews.llvm.org/D18706