Re: [Mesa-dev] [PATCH] radeonsi: Don't use global variables for tess lds

2016-08-29 Thread Marek Olšák
On Mon, Aug 29, 2016 at 2:21 AM, Edward O'Callaghan wrote: > Missing Signoff-by line but otherwise with that fix, Signed-off-by isn't required in Mesa. The patch is: Reviewed-by: Marek Olšák Marek > > Reviewed-By: Edward O'Callaghan > > On 08/27/2016 05:52 AM, Tom Stellard wrote: >> We were

Re: [Mesa-dev] [PATCH] radeonsi: Don't use global variables for tess lds

2016-08-28 Thread Edward O'Callaghan
Missing Signoff-by line but otherwise with that fix, Reviewed-By: Edward O'Callaghan On 08/27/2016 05:52 AM, Tom Stellard wrote: > We were allocating global variables for the maximum LDS size > which made the compiler think we were using all of LDS, which > isn't the case. > --- > src/gallium/d

[Mesa-dev] [PATCH] radeonsi: Don't use global variables for tess lds

2016-08-26 Thread Tom Stellard
We were allocating global variables for the maximum LDS size which made the compiler think we were using all of LDS, which isn't the case. --- src/gallium/drivers/radeonsi/si_shader.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/src/gallium/drivers/radeonsi/s