Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-15 Thread Emil Velikov
On 11 March 2017 at 16:45, Matt Turner wrote: >>> AR="gcc-ar" >>> NM="gcc-nm" >>> RANLIB="gcc-ranlib" >>> >> One should not need these. Have you checked that the issue is >> reproducible w/o them ? > > I think they are necessary > > https://gcc.gnu.org/ml/gcc-help/2012-03/msg00100.html > > (thoug

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-13 Thread Emil Velikov
On 12 March 2017 at 13:30, Steven Newbury wrote: > On Sun, 2017-03-12 at 12:25 +, Mike Lothian wrote: >> For what its worth I compile Mesa using LTO with the gold linker so I >> don't need to specify -fuse-linker-plugin - everything works fine >> here >> >> CFLAGS="-O3 -march=skylake -pipe -fl

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-12 Thread Steven Newbury
On Sun, 2017-03-12 at 12:25 +, Mike Lothian wrote: > For what its worth I compile Mesa using LTO with the gold linker so I > don't need to specify -fuse-linker-plugin - everything works fine > here > > CFLAGS="-O3 -march=skylake -pipe -flto=8"  > CXXFLAGS="${CFLAGS}"  > LDFLAGS="-Wl,-O1 -Wl,--

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-12 Thread Mike Lothian
For what its worth I compile Mesa using LTO with the gold linker so I don't need to specify -fuse-linker-plugin - everything works fine here CFLAGS="-O3 -march=skylake -pipe -flto=8" CXXFLAGS="${CFLAGS}" LDFLAGS="-Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed -Wl,-flto=8" AR="gcc-ar" NM="gcc-nm" RA

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-11 Thread Matt Turner
On Sat, Mar 11, 2017 at 7:31 AM, Emil Velikov wrote: > On 10 March 2017 at 13:36, Steven Newbury wrote: >> On Fri, 2017-03-10 at 12:11 +, Emil Velikov wrote: >>> On 9 March 2017 at 14:39, Steven Newbury >>> wrote: >>> > Introduction of zlib compression for the shader cache means >>> > zlib n

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-11 Thread Emil Velikov
On 10 March 2017 at 13:36, Steven Newbury wrote: > On Fri, 2017-03-10 at 12:11 +, Emil Velikov wrote: >> On 9 March 2017 at 14:39, Steven Newbury >> wrote: >> > Introduction of zlib compression for the shader cache means >> > zlib needs to be explicitly linked to libOSMesa and libstandalone >

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-10 Thread Steven Newbury
On Fri, 2017-03-10 at 12:11 +, Emil Velikov wrote: > On 9 March 2017 at 14:39, Steven Newbury > wrote: > > Introduction of zlib compression for the shader cache means > > zlib needs to be explicitly linked to libOSMesa and libstandalone > > otherwise build fails when LTO is used. > > --- > >

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-10 Thread Emil Velikov
On 9 March 2017 at 14:39, Steven Newbury wrote: > Introduction of zlib compression for the shader cache means > zlib needs to be explicitly linked to libOSMesa and libstandalone > otherwise build fails when LTO is used. > --- How exactly are you doing the LTO build ? The only user of ZLIB is libm

Re: [Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-10 Thread Marek Olšák
Acked-by: Marek Olšák Marek On Thu, Mar 9, 2017 at 3:39 PM, Steven Newbury wrote: > Introduction of zlib compression for the shader cache means > zlib needs to be explicitly linked to libOSMesa and libstandalone > otherwise build fails when LTO is used. > --- > src/compiler/Makefile.glsl.am

[Mesa-dev] [PATCH] util/disk_cache: fix zlib linking with LTO build

2017-03-09 Thread Steven Newbury
Introduction of zlib compression for the shader cache means zlib needs to be explicitly linked to libOSMesa and libstandalone otherwise build fails when LTO is used. ---  src/compiler/Makefile.glsl.am  | 1 +  src/gallium/targets/osmesa/Makefile.am | 3 ++-  2 files changed, 3 insertions(+),