[patch] Copy dll storage in copyAttributes

[Reid Kleckner]

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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140212/1e0979ba/attachment.html>