jemalloc and clang (was: Re: svn commit: r239462 - in head)
Jason Evans
jasone at freebsd.org
Wed Aug 22 16:18:19 UTC 2012
On Aug 21, 2012, at 1:39 AM, Dimitry Andric <dim at FreeBSD.org> wrote:
> On 2012-08-21 02:17, Jan Beich wrote:
> ...
>> Time to revert r228540?
>>
>> Index: contrib/jemalloc/include/jemalloc/jemalloc_FreeBSD.h
>> ===================================================================
>> --- contrib/jemalloc/include/jemalloc/jemalloc_FreeBSD.h (revision 239467)
>> +++ contrib/jemalloc/include/jemalloc/jemalloc_FreeBSD.h (working copy)
>> @@ -56,10 +56,6 @@
>> #ifndef JEMALLOC_TLS_MODEL
>> # define JEMALLOC_TLS_MODEL /* Default. */
>> #endif
>> -#ifdef __clang__
>> -# undef JEMALLOC_TLS_MODEL
>> -# define JEMALLOC_TLS_MODEL /* clang does not support tls_model yet. */
>> -#endif
>>
>> #define STATIC_PAGE_SHIFT PAGE_SHIFT
>> #define LG_SIZEOF_INT 2
>
> Well, if Jason would like to support upstream jemalloc for different
> versions of clang, it is probably better to to use __has_feature()
> instead.
>
> Jason, what do you think of the attached patch? Or could we just remove
> the whole #ifdef __clang__ part?
> <jemalloc-clang-tlsmodel-1.diff>
I'm happy with the above patch (completely removing the #ifdef __clang__).
Jason
More information about the svn-src-head
mailing list