[Mesa-dev] [PATCH 2/2] radeonsi: remove constant index limitation v2

Michel Dänzer michel at daenzer.net
Tue Feb 5 08:00:58 PST 2013


On Die, 2013-02-05 at 15:46 +0100, Christian König wrote: 
> From: Christian König <christian.koenig at amd.com>
> 
> With the llvm patches, fixing 14 piglit tests in total.
> 
> v2: increase the const limit
> 
> Signed-off-by: Christian König <christian.koenig at amd.com>
> ---
>  src/gallium/drivers/radeonsi/radeonsi_pipe.c   |    2 +-
>  src/gallium/drivers/radeonsi/radeonsi_shader.c |    6 ------
>  2 files changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/src/gallium/drivers/radeonsi/radeonsi_pipe.c b/src/gallium/drivers/radeonsi/radeonsi_pipe.c
> index 744dac4..5394f5d 100644
> --- a/src/gallium/drivers/radeonsi/radeonsi_pipe.c
> +++ b/src/gallium/drivers/radeonsi/radeonsi_pipe.c
> @@ -456,7 +456,7 @@ static int r600_get_shader_param(struct pipe_screen* pscreen, unsigned shader, e
>  		/* FIXME Isn't this equal to TEMPS? */
>  		return 1; /* Max native address registers */
>  	case PIPE_SHADER_CAP_MAX_CONSTS:
> -		return 64;
> +		return 4096;

Where does the value 4096 come from? Maybe add a comment.

Other than that, this series is

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>

Also, this series and the LLVM series are

Tested-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer


More information about the mesa-dev mailing list