Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-19 Thread Emil Velikov
On 19 October 2016 at 08:58, Michel Dänzer wrote: > On 18/10/16 11:39 PM, Emil Velikov wrote: >> On 17 October 2016 at 08:56, Michel Dänzer wrote: >>> On 14/10/16 07:02 PM, Emil Velikov wrote: >>> * src/gallium/drivers/radeon/r600_pipe_common.c

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-19 Thread Michel Dänzer
On 18/10/16 11:39 PM, Emil Velikov wrote: > On 17 October 2016 at 08:56, Michel Dänzer wrote: >> On 14/10/16 07:02 PM, Emil Velikov wrote: >> >>> * src/gallium/drivers/radeon/r600_pipe_common.c >>> No actual bug, yet misleading. >> >> If you want to call it that... I'd say

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-18 Thread Emil Velikov
On 17 October 2016 at 08:56, Michel Dänzer wrote: > On 14/10/16 07:02 PM, Emil Velikov wrote: >> On 14 October 2016 at 09:45, Michel Dänzer wrote: >>> On 14/10/16 05:14 PM, Emil Velikov wrote: On 14 October 2016 at 01:45, Michel Dänzer

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-17 Thread Michel Dänzer
On 14/10/16 07:02 PM, Emil Velikov wrote: > On 14 October 2016 at 09:45, Michel Dänzer wrote: >> On 14/10/16 05:14 PM, Emil Velikov wrote: >>> On 14 October 2016 at 01:45, Michel Dänzer wrote: On 13/10/16 07:14 PM, Emil Velikov wrote: > On 13

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-14 Thread Emil Velikov
On 14 October 2016 at 09:45, Michel Dänzer wrote: > On 14/10/16 05:14 PM, Emil Velikov wrote: >> On 14 October 2016 at 01:45, Michel Dänzer wrote: >>> On 13/10/16 07:14 PM, Emil Velikov wrote: On 13 October 2016 at 04:07, Michel Dänzer

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-14 Thread Michel Dänzer
On 14/10/16 05:14 PM, Emil Velikov wrote: > On 14 October 2016 at 01:45, Michel Dänzer wrote: >> On 13/10/16 07:14 PM, Emil Velikov wrote: >>> On 13 October 2016 at 04:07, Michel Dänzer wrote: On 13/10/16 03:37 AM, Tobias Droste wrote: > Am

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-14 Thread Emil Velikov
On 14 October 2016 at 01:45, Michel Dänzer wrote: > On 13/10/16 07:14 PM, Emil Velikov wrote: >> On 13 October 2016 at 04:07, Michel Dänzer wrote: >>> On 13/10/16 03:37 AM, Tobias Droste wrote: Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-13 Thread Michel Dänzer
On 14/10/16 09:54 AM, Michel Dänzer wrote: > On 13/10/16 11:16 PM, Mike Lothian wrote: >> I think there were some issues with some programs / games that had >> clashing symbols but I'm not sure if that's still an issue > > I'm only aware of such issues with the Steam runtime, which are arguably >

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-13 Thread Michel Dänzer
On 13/10/16 11:16 PM, Mike Lothian wrote: > If you recompile llvm and mesa each time, are there any downsides of > using the shared libs? No particular downsides come to mind, rather upsides: * Generally no requirement to rebuild Mesa after changes to LLVM. * Both LLVM and Mesa take

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-13 Thread Michel Dänzer
On 13/10/16 07:14 PM, Emil Velikov wrote: > On 13 October 2016 at 04:07, Michel Dänzer wrote: >> On 13/10/16 03:37 AM, Tobias Droste wrote: >>> Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil Velikov: > > +LLVM_VERSION_REQUIRED_OPENCL=3.6.0 >

[Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-13 Thread Emil Velikov
On Thursday, 13 October 2016, Mike Lothian > wrote: > If you recompile llvm and mesa each time, are there any downsides of using > the shared libs? > > How many devs rebuild package X when one of its dependencies has a bug

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-13 Thread Mike Lothian
If you recompile llvm and mesa each time, are there any downsides of using the shared libs? I think there were some issues with some programs / games that had clashing symbols but I'm not sure if that's still an issue On Thu, 13 Oct 2016 at 11:15 Emil Velikov wrote: >

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-13 Thread Emil Velikov
On 13 October 2016 at 04:07, Michel Dänzer wrote: > On 13/10/16 03:37 AM, Tobias Droste wrote: >> Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil Velikov: +LLVM_VERSION_REQUIRED_OPENCL=3.6.0 +LLVM_VERSION_REQUIRED_R600=3.6.0

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-12 Thread Michel Dänzer
On 13/10/16 03:37 AM, Tobias Droste wrote: > Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil Velikov: >>> >>> +LLVM_VERSION_REQUIRED_OPENCL=3.6.0 >>> +LLVM_VERSION_REQUIRED_R600=3.6.0 >>> +LLVM_VERSION_REQUIRED_RADEONSI=3.6.0 >> >> There's a small related gotcha: as-is at build time we

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-12 Thread Emil Velikov
On 12 October 2016 at 19:37, Tobias Droste wrote: > Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil Velikov: >> > >> > +dnl LLVM versions >> > +LLVM_VERSION_REQUIRED_GALLIUM=3.3.0 >> > +LLVM_VERSION_REQUIRED_LLVMPIPE=3.6.0 >> >> Not exactly sure why/how llvmpipe got 3.6

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-12 Thread Tobias Droste
Am Mittwoch, 12. Oktober 2016, 11:53:50 CEST schrieb Emil Velikov: > > > > +dnl LLVM versions > > +LLVM_VERSION_REQUIRED_GALLIUM=3.3.0 > > +LLVM_VERSION_REQUIRED_LLVMPIPE=3.6.0 > > Not exactly sure why/how llvmpipe got 3.6 since the driver reuses the > "gallium" one. Yes you're right. I'm going

Re: [Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-12 Thread Emil Velikov
On 12 October 2016 at 00:02, Tobias Droste wrote: > Consolidate the required LLVM versions at the top where the other > versions for dependencies are listed. > Rework the LLVM_VERSION and LLVM_VERSION_INT calculation to make the > format "x.y.z." possible. > Not sure how others

[Mesa-dev] [PATCH v3 25/25] configure.ac: Add required LLVM versions to the top

2016-10-11 Thread Tobias Droste
Consolidate the required LLVM versions at the top where the other versions for dependencies are listed. Rework the LLVM_VERSION and LLVM_VERSION_INT calculation to make the format "x.y.z." possible. LLVM_VERSION_INT is now including LLVM_VERSION_PATCH. For the C define HAVE_LLVM is set now which