[LLVMdev] [PATCH] TLS support for Windows 32+64bit

Kai kai at redstar.de
Mon Jan 9 09:06:39 PST 2012


Hi!

Thanks for your answer!
I reworked the patch: the tabs and the commented-out is gone. I also 
tried to limit line-length to 80 chars. Applies cleanly to r147775 now.

Kai

On 01.01.2012 22:01, Eli Friedman wrote:
> On Sun, Jan 1, 2012 at 10:44 AM, Kai<kai at redstar.de>  wrote:
>> Happy new year to all!
>>
>> The attached patch adds TLS support for x86_64-pc-win32 and x86-pc-win32.
>> Implemented is the implicit TLS model (__declspec(thread) in Visual C++).
>> Please review. Thanks!
>
> Rough review: looks like the patch is in the right direction; please
> take out all tabs, please take out all commented-out code.
>
> -Eli

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: tls20120109.diff
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120109/63295641/attachment.ksh>


More information about the llvm-dev mailing list