Re: [Mesa-dev] [PATCH] st/nir: fix atomic lowering for gallium drivers

2018-03-19 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Mon, Mar 19, 2018 at 8:32 AM, Timothy Arceri wrote: > i965 and gallium handle the atomic buffer index differently. It was > just by luck that the single piglit test for this was passing. > > For gallium we use the atomic binding so that we match the handling

[Mesa-dev] [PATCH] st/nir: fix atomic lowering for gallium drivers

2018-03-19 Thread Timothy Arceri
i965 and gallium handle the atomic buffer index differently. It was just by luck that the single piglit test for this was passing. For gallium we use the atomic binding so that we match the handling in st_bind_atomics(). On radeonsi this fixes the CTS test: KHR-GL43.shader_storage_buffer_object.a