Siarhei Siamashka <siarhei.siamas...@gmail.com> writes:

>> > I would suggest to just change the order of checks for
>> > __declspec(thread) and __thread in configure.ac
>> 
>> We should maybe try using -Werror=ignored-attributes in the test, or
>> just prefer the first one.
>
> Using -Werror=ignored-attributes will not help because clang-3.1 (and
> maybe older versions too) just silently ignores __declspec(thread).
> This non-working TLS problem could have been detected by the pixman test
> suite if clang had OpenMP support. In any case, I was just lucky to
> notice the fresh warning when doing preemptive testing for the current
> clang svn (just to be sure that pixman would not need any ugly compiler
> bug workarounds for the upcoming clang-3.2).
>
> Regarding your original patch, how does iOS behave when somebody
> tries to use __thread keyword? Does it fail compilation?

If nobody knows the answer to this, let's just change the order of the
checks as you suggested. We can't support iOS unless someone is willing
to maintain it.


Soren
_______________________________________________
Pixman mailing list
Pixman@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pixman

Reply via email to