[patch] Copy dll storage in copyAttributes

[Rafael Espíndola]

r201295. Thanks.

On 12 February 2014 22:20, Reid Kleckner <rnk at google.com> wrote:
> Yeah, we should definitely copy it.  Consider MergeFunctions, which merges
> two functions and then creates a thunk with attributes of the first.  We
> definitely want the new thunk to be exported if we do that.
>
>
> On Wed, Feb 12, 2014 at 6:47 PM, Rafael Espíndola
> <rafael.espindola at gmail.com> wrote:
>>
>> The attached patch updates copyAttributes to also copy the dll
>> storage. I am not familiar with dll storage class, so please check if
>> this is the desired behavior from llvm-link. If not, I will write a
>> test showing that we *don't* copy it :-)
>>
>> Cheers,
>> Rafael
>
>