On 26 June 2012 13:40, Hans Wennborg <[email protected]> wrote:
> Hi all,
>
> The attached patch adds support for the -ftls-model command-line flag.
> Please take a look.

There was some work at unifying Options.td and CC1Options.td. Do we
have to change both?

Why does

llvm::GlobalVariable::ThreadLocalMode GetLLVMTLSModel(StringRef S)

need DefaultCase?

Can you use a switch over Opts.getDefaultTLSModel in
CompilerInvocation.cpp and a StringSwitch over the Model?

> Thanks,
> Hans
>

Cheers,
Rafael
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to