Re: [Mesa-dev] [RFC] Fix for clang compiler issue as reported in Bug 91826

2015-09-01 Thread Serge Martin (EdB)
On Tuesday 01 September 2015 17:10:33 Albert Freeman wrote: > Clang tryed to declare the non type member of struct module (enum type type) > (in clover/core/module.hpp) instead of a variable of type enum (enum type). > > Signed-off-by: Albert Freeman Reviewed by Serge Martin Can it be pushed t

Re: [Mesa-dev] [PATCH] clover: fix llvm 3.5 build error

2015-08-21 Thread Serge Martin (EdB)
upport won't be backport to llvm 3.5 this is good for me. And it no longer make clover crash. Reviewed-by: Serge Martin (EdB) But since I don't have commit access, not sure what my r-b worth. Could it be pushed before the branch point? > --- > .../state_trackers/clove

Re: [Mesa-dev] [PATCH] clover: fix llvm 3.5 build error

2015-08-16 Thread Serge Martin (EdB)
On Sunday 09 August 2015 11:52:13 orome wrote: > On Sun, 2015-08-09 at 15:39 +0200, Zoltan Gilian wrote: > > There is no MDOperand in llvm 3.5. > > > > v2: Check if kernel metadata is present to avoid crash (EdB). > > still builds for me. Sadly, it's still crashing for me. auto kernel_node = ge

Re: [Mesa-dev] [PATCH] clover: block on transfer api calls when requested

2015-08-15 Thread Serge Martin (EdB)
Hello This looks like what I've send a few weeks ago [0]. It's seems we should come with something a little different as explain by curro in [1] [0] http://lists.freedesktop.org/archives/mesa-dev/2015-August/090479.html [1] http://lists.freedesktop.org/archives/mesa-dev/2015-June/086110.html

[Mesa-dev] [PATCH v4] clover: stub missing CL 1.2 functions

2015-08-07 Thread Serge Martin (EdB)
As sugested by Tom a long time ago and in order to be able to create Piglit tests v2: replace NOT_SUPPORTED_BY_CL_1_1 macro with an inline function remove extra space in clLinkProgram arg v3: use __func__ v4: back to a macro, it make more sense to use it with __func__ --- src/gallium/state_trac