Re: [Mesa-dev] [PATCH] Fix warnings.

2016-05-29 Thread Vedran Miletic
On 05/29/2016 01:54 PM, Romain Failliot wrote: Can now compile with -Werror. Fixed -Wunused-variable, -Wunused-but-set-variable, -Wmaybe-uninitialized and -Wunused-function. To prevent warnings for variables only used in asserts, I void them right after the call to assert(). The changes look

Re: [Mesa-dev] [PATCH 2/2] Add LLVM version to Mesa version strings

2016-05-23 Thread Vedran Miletic
On 05/23/2016 09:21 AM, Giuseppe Bilotta wrote: Code generation (kernel compilation) may sometimes hit LLVM-specific bugs. Adding the used LLVM version to the version string may make bug triaging easier. (This was inspired by a similar patch recently proposed for pocl.) Signed-off-by: Giuseppe

Re: [Mesa-dev] [PATCH 2/2] Add LLVM version to Mesa version strings

2016-05-23 Thread Vedran Miletic
On 05/24/2016 12:46 AM, Marek Olšák wrote: On Mon, May 23, 2016 at 10:47 PM, Giuseppe Bilotta wrote: On Mon, May 23, 2016 at 4:45 PM, Marek Olšák wrote: I think it would be better to put stuff like this only into the renderer string of drivers

Re: [Mesa-dev] [RFC PATCH] clover: add LLVM version to device and platform version

2016-05-22 Thread Vedran Miletic
On 05/22/2016 10:13 PM, Giuseppe Bilotta wrote: On Sun, May 22, 2016 at 1:08 PM, Serge Martin wrote: Mesa or MESA spelling shouldn't be an issue. And that would be coherent with CL_PLATFORM_VENDOR that is returning "Mesa" Excellent. I'll unify the version string suffix