Re: [Mesa-dev] Fix __atomic* builtins detection for Clang

2017-05-14 Thread Dimitry Andric
On 13 May 2017, at 09:01, Jan Beich wrote: > > Jonathan Gray writes: > >> On Sat, May 13, 2017 at 02:22:30AM +0200, Jan Beich wrote: >> >>> "int" isn't large enough and lack of builtins only manifests at link time. >>> This was breaking build on FreeBSD 11.0 i386 with Clang 3.8.0. >>> >>> gls

Re: [Mesa-dev] Fix __atomic* builtins detection for Clang

2017-05-13 Thread Jonathan Gray
On Sat, May 13, 2017 at 09:01:49AM +0200, Jan Beich wrote: > Jonathan Gray writes: > > > On Sat, May 13, 2017 at 02:22:30AM +0200, Jan Beich wrote: > > > >> "int" isn't large enough and lack of builtins only manifests at link time. > >> This was breaking build on FreeBSD 11.0 i386 with Clang 3.8.

Re: [Mesa-dev] Fix __atomic* builtins detection for Clang

2017-05-13 Thread Jan Beich
Jan Beich writes: > "int" isn't large enough and lack of builtins only manifests at link time. > This was breaking build on FreeBSD 11.0 i386 with Clang 3.8.0. > > glsl/.libs/libstandalone.a(libmesautil_la-disk_cache.o): In function > `disk_cache_remove': > disk_cache.c:(.text+0x763): undefined

Re: [Mesa-dev] Fix __atomic* builtins detection for Clang

2017-05-13 Thread Jan Beich
Jonathan Gray writes: > commit a6a38a038bd62e6d9558905f00bef81b5e7e6fcc > Author: Grazvydas Ignotas > Date: Fri Mar 31 01:26:25 2017 +0300 > > util/u_atomic: provide 64bit atomics where they're missing > > attempts to handle powerpc like situations, though not sync > builtins being missing

Re: [Mesa-dev] Fix __atomic* builtins detection for Clang

2017-05-13 Thread Jan Beich
Jonathan Gray writes: > On Sat, May 13, 2017 at 02:22:30AM +0200, Jan Beich wrote: > >> "int" isn't large enough and lack of builtins only manifests at link time. >> This was breaking build on FreeBSD 11.0 i386 with Clang 3.8.0. >> >> glsl/.libs/libstandalone.a(libmesautil_la-disk_cache.o): In f

Re: [Mesa-dev] Fix __atomic* builtins detection for Clang

2017-05-12 Thread Jonathan Gray
On Sat, May 13, 2017 at 02:22:30AM +0200, Jan Beich wrote: > "int" isn't large enough and lack of builtins only manifests at link time. > This was breaking build on FreeBSD 11.0 i386 with Clang 3.8.0. You should be building with -march=i586 on i386 for CX8. The diff is wrong as it will break othe

[Mesa-dev] Fix __atomic* builtins detection for Clang

2017-05-12 Thread Jan Beich
"int" isn't large enough and lack of builtins only manifests at link time. This was breaking build on FreeBSD 11.0 i386 with Clang 3.8.0. glsl/.libs/libstandalone.a(libmesautil_la-disk_cache.o): In function `disk_cache_remove': disk_cache.c:(.text+0x763): undefined reference to `__atomic_fetch_ad