[llvm-commits] [llvm] r59208 - /llvm/trunk/lib/Transforms/IPO/StripSymbols.cpp

[Chris Lattner]

On Nov 18, 2008, at 11:12 AM, Devang Patel wrote:

>
> On Nov 17, 2008, at 5:00 PM, Chris Lattner wrote:
>
>> On Nov 17, 2008, at 4:45 PM, Devang Patel wrote:
>>>> I don't think that matching on the names is really a sound way to
>>>> go.
>>>> How about just looping over all the globals, removing dead globals
>>>> in
>>>> the llvm.metadata section?
>>>
>>>
>>> If it start remove dead globals then it becomes difficult to compare
>>> "-
>>> strip-debug -std-compile-opts" output with "-std-compile-opts - 
>>> strip-
>>> debug" output.
>>
>> What other dead metadata can exist?
>
>
> llvm.used  uses llvm.metadata. It is possible that symbols marked as
> attribute used does not have any use. In such cases we want to
> preserve them. Until alternative mechanism is available, name matching
> will be required anyway.

Those don't have internal linkage.

-Chris