[Mesa-dev] [PATCH] radeonsi: Allow longer intrinsic names

Tom Stellard tom at stellard.net
Tue Oct 29 17:48:20 CET 2013


On Sun, Oct 27, 2013 at 07:36:07PM +0100, Kai Wasserbäch wrote:
> Fixes a boat load of Piglit tests for me, which crashed like fdo#70913
> before.
> 
> Thanks to Michel Dänzer for the tip.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70913
> Signed-off-by: Kai Wasserbäch <kai at dev.carbon-project.org>

This looks OK to me, which intrinsic names were too long?

Reviewed-by: Tom Stellard <thomas.stellard at amd.com>

> ---
>  src/gallium/drivers/radeonsi/radeonsi_shader.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c b/src/gallium/drivers/radeonsi/radeonsi_shader.c
> index 9f81a7b..dff8be0 100644
> --- a/src/gallium/drivers/radeonsi/radeonsi_shader.c
> +++ b/src/gallium/drivers/radeonsi/radeonsi_shader.c
> @@ -1425,7 +1425,7 @@ static void build_tex_intrinsic(const struct lp_build_tgsi_action * action,
>  				struct lp_build_emit_data * emit_data)
>  {
>  	struct lp_build_context * base = &bld_base->base;
> -	char intr_name[23];
> +	char intr_name[127];
>  
>  	sprintf(intr_name, "%sv%ui32", action->intr_name,
>  		LLVMGetVectorSize(LLVMTypeOf(emit_data->args[0])));
> -- 
> 1.8.4.rc3
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list