[PATCH] Bitrig uses TLS

[Reid Kleckner]

I wouldn't test it, since the codepaths are covered by other targets.  You
can drop the assignment entirely.  The default value of TLSSupported is
true.


On Thu, May 1, 2014 at 1:49 PM, Patrick Wildt <mail at patrick-wildt.de> wrote:

> Hi,
>
> Bitrig now does TLS, so it should be marked as supported in the
> TargetInfo.
>
> Does this patch need a test?  Is there some test I can add
> a Bitrig target to?
>
> \Patrick
>
> diff --git a/lib/Basic/Targets.cpp b/lib/Basic/Targets.cpp
> index 6fdd0d0..387e3d9 100644
> --- a/lib/Basic/Targets.cpp
> +++ b/lib/Basic/Targets.cpp
> @@ -445,7 +445,7 @@ protected:
>  public:
>    BitrigTargetInfo(const llvm::Triple &Triple) :
> OSTargetInfo<Target>(Triple) {
>      this->UserLabelPrefix = "";
> -    this->TLSSupported = false;
> +    this->TLSSupported = true;
>      this->MCountName = "__mcount";
>    }
>  };
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140501/49125de7/attachment.html>