[llvm-commits] [LLVMdev] [Patch, RFC] Re: Adding support for explicitly specified TLS models (PR9788)

[Rafael Espíndola]

On 4 June 2012 10:49, Hans Wennborg <hans at chromium.org> wrote:
> Reviving this thread with a patch!
>
> And some comments inline.
>
> Please take a look and let me know what you think.

Just a high level comment, why do you need the 4 modes + default?
Can't clang just produce globaldynamic (or the attribute value) and
let llvm optimize it? Since in the end the linker is allowed to relax
tls access too, llvm should be allowed to make the mode more specific
even if the user added an attribute at the C/C++ level.

>  - Hans

Cheers,
Rafael