[cfe-commits] [Patch] Add -ftls-model command-line flag (pr9788)

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Jun 27 07:30:57 PDT 2012


On 26 June 2012 13:40, Hans Wennborg <hans at chromium.org> 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



More information about the cfe-commits mailing list