r207470 - LLVM supports TLS on Windows and we can use it from Clang

Reid Kleckner reid at kleckner.net
Mon Apr 28 17:11:30 PDT 2014


Author: rnk
Date: Mon Apr 28 19:11:30 2014
New Revision: 207470

URL: http://llvm.org/viewvc/llvm-project?rev=207470&view=rev
Log:
LLVM supports TLS on Windows and we can use it from Clang

Patch by Martell Malone!

Differential Revision: http://reviews.llvm.org/D3421

Modified:
    cfe/trunk/lib/Basic/Targets.cpp
    cfe/trunk/test/Sema/tls.c

Modified: cfe/trunk/lib/Basic/Targets.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Targets.cpp?rev=207470&r1=207469&r2=207470&view=diff
==============================================================================
--- cfe/trunk/lib/Basic/Targets.cpp (original)
+++ cfe/trunk/lib/Basic/Targets.cpp Mon Apr 28 19:11:30 2014
@@ -3051,7 +3051,6 @@ class WindowsX86_32TargetInfo : public W
 public:
   WindowsX86_32TargetInfo(const llvm::Triple &Triple)
       : WindowsTargetInfo<X86_32TargetInfo>(Triple) {
-    TLSSupported = false;
     WCharType = UnsignedShort;
     DoubleAlign = LongLongAlign = 64;
     DescriptionString = "e-m:w-p:32:32-i64:64-f80:32-n8:16:32-S32";
@@ -3287,7 +3286,6 @@ class WindowsX86_64TargetInfo : public W
 public:
   WindowsX86_64TargetInfo(const llvm::Triple &Triple)
       : WindowsTargetInfo<X86_64TargetInfo>(Triple) {
-    TLSSupported = false;
     WCharType = UnsignedShort;
     LongWidth = LongAlign = 32;
     DoubleAlign = LongLongAlign = 64;

Modified: cfe/trunk/test/Sema/tls.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/tls.c?rev=207470&r1=207469&r2=207470&view=diff
==============================================================================
--- cfe/trunk/test/Sema/tls.c (original)
+++ cfe/trunk/test/Sema/tls.c Mon Apr 28 19:11:30 2014
@@ -9,9 +9,8 @@
 // RUN: not %clang_cc1 -triple x86_64-apple-darwin10 -fsyntax-only %s
 // RUN: %clang_cc1 -triple x86_64-apple-macos10.7.0 -fsyntax-only %s
 
-// FIXME: I thought it was supported actually?
-// RUN: not %clang_cc1 -triple x86_64-pc-win32 -fsyntax-only %s
-// RUN: not %clang_cc1 -triple i386-pc-win32 -fsyntax-only %s
+// RUN: %clang_cc1 -triple x86_64-pc-win32 -fsyntax-only %s
+// RUN: %clang_cc1 -triple i386-pc-win32 -fsyntax-only %s
 
 // OpenBSD does not suppport TLS.
 // RUN: not %clang_cc1 -triple x86_64-pc-openbsd -fsyntax-only %s





More information about the cfe-commits mailing list