Re: [Mesa-dev] [PATCH 2/2] ac/nir: set GLC=1 for load/store of coherent/volatile images

2018-02-22 Thread Samuel Pitoiset
On 02/22/2018 11:45 AM, Bas Nieuwenhuizen wrote: Ideally we find a way to also do it for buffers, but for this series I didn't mention it in the commit message, but yes we should. Reviewed-by: Bas Nieuwenhuizen > On Thu, Feb 22, 2018 at 10:25 AM, Samuel P

Re: [Mesa-dev] [PATCH 2/2] ac/nir: set GLC=1 for load/store of coherent/volatile images

2018-02-22 Thread Bas Nieuwenhuizen
Ideally we find a way to also do it for buffers, but for this series Reviewed-by: Bas Nieuwenhuizen On Thu, Feb 22, 2018 at 10:25 AM, Samuel Pitoiset wrote: > This disables persistence accross wavefronts. > > F1 2017 and Wolfenstein 2 appear to use some coherent images > but this patch doesn't

[Mesa-dev] [PATCH 2/2] ac/nir: set GLC=1 for load/store of coherent/volatile images

2018-02-22 Thread Samuel Pitoiset
This disables persistence accross wavefronts. F1 2017 and Wolfenstein 2 appear to use some coherent images but this patch doesn't seem to change anything. Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_nir_to_llvm.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git