Re: [Mesa-dev] [PATCH] glsl: Disable tree grafting optimization for shared variables

2016-01-22 Thread Iago Toral
Right, we had to do the same for buffer variables to prevent losing writes to the underlying buffer. Reviewed-by: Iago Toral Quiroga On Thu, 2016-01-21 at 16:47 -0800, Jordan Justen wrote: > Fixes: dEQP-GLES31.functional.compute.basic.shared_atomic_op_multiple_groups > > From: https://android.g

Re: [Mesa-dev] [PATCH] glsl: Disable tree grafting optimization for shared variables

2016-01-21 Thread Samuel Pitoiset
Hi Jordan, Thanks for the patch, it also fixes the following dEQP tests (which are related to the multiple groups one): dEQP-GLES31.functional.compute.basic.shared_atomic_op_multiple_invocation dEQP-GLES31.functional.compute.basic.shared_atomic_op_single_group dEQP-GLES31.functional.compute.ba

[Mesa-dev] [PATCH] glsl: Disable tree grafting optimization for shared variables

2016-01-21 Thread Jordan Justen
Fixes: dEQP-GLES31.functional.compute.basic.shared_atomic_op_multiple_groups From: https://android.googlesource.com/platform/external/deqp Reported-by: Ilia Mirkin Signed-off-by: Jordan Justen --- src/glsl/opt_tree_grafting.cpp | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-)