[Mesa-dev] [PATCH 2/2 v3] add visibility hidden to tls entry points

[Sedat Dilek]

On Tue, Feb 17, 2015 at 12:38 PM, Marc Dietrich <marvin24 at gmx.de> wrote:
> Am Dienstag, 17. Februar 2015, 11:58:06 schrieb Sedat Dilek:
>> On Tue, Feb 17, 2015 at 10:40 AM, Marc Dietrich <marvin24 at gmx.de> wrote:
>> > Avoid redefined symbol errors in clang. Based on a suggestion from
>> > Rafael Ávila de Espíndola <rafael.espindola at gmail.com> in
>> > http://llvm.org/bugs/show_bug.cgi?id=19778.
>> >
>> > Signed-off-by: Marc Dietrich <marvin24 at gmx.de>
>> > ---
>> > v2: - no change
>> > v3: - include util directory in Makefile.am in order to avoid relative
>> >
>> >       include paths,
>> >
>> >     - make tls entry points array const
>>
>> I adapted this patch as v4 to fit mesa v10.4.4, but it fails (see
>> log-file in attached tarball).
>
> the configure output says that visibility is not supported by your clang.
> Interestingly, it is supported here (also clang 3.6 from yesterday). What does
> config.log say about it?
>

Here is my config.log.

$ grep -i visibility config.log
configure:18401: checking for __attribute__((visibility))
ax_cv_have_func_attribute_visibility=no
VISIBILITY_CFLAGS=''
VISIBILITY_CXXFLAGS=''

- Sedat -
-------------- next part --------------
A non-text attachment was scrubbed...
Name: config.log
Type: text/x-log
Size: 325727 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150217/61facd8b/attachment-0001.bin>