Re: [Mesa-dev] [PATCH 4/7] radeonsi: make use of LOAD for UBOs

2017-09-14 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Tue, Aug 22, 2017 at 2:14 PM, Timothy Arceri wrote: > v2: always set can_speculate and allow_smem to true > --- > src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 31 > +++ > 1 file changed, 21 insertions(+), 10 deletions(-) > > diff --

Re: [Mesa-dev] [PATCH 4/7] radeonsi: make use of LOAD for UBOs

2017-08-29 Thread Timothy Arceri
On 30/08/17 09:39, Marek Olšák wrote: Did you check shader-db for differences in code generation? Yes, for more details and discussion please see the cover letter. Thanks, Tim Marek On Tue, Aug 22, 2017 at 2:14 PM, Timothy Arceri wrote: v2: always set can_speculate and allow_smem to true

Re: [Mesa-dev] [PATCH 4/7] radeonsi: make use of LOAD for UBOs

2017-08-29 Thread Marek Olšák
Did you check shader-db for differences in code generation? Marek On Tue, Aug 22, 2017 at 2:14 PM, Timothy Arceri wrote: > v2: always set can_speculate and allow_smem to true > --- > src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 31 > +++ > 1 file changed, 21 insertio

[Mesa-dev] [PATCH 4/7] radeonsi: make use of LOAD for UBOs

2017-08-22 Thread Timothy Arceri
v2: always set can_speculate and allow_smem to true --- src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 31 +++ 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c b/src/gallium/drivers/radeonsi/si_shader_tgsi_m