Chad Versace <chad.vers...@linux.intel.com> writes:

> The bug: The printed horizontal stride was the numerical value of the
>   BRW_HORIZONTAL_$N enum.
> The fix: Translate the enum before printing.
>
> Note: This is a candidate for the stable releases.
> Signed-off-by: Chad Versace <chad.vers...@linux.intel.com>

> @@ -537,8 +539,8 @@ static int dest (FILE *file, struct brw_instruction *inst)
>                                       
> reg_type_size[inst->bits1.ia1.dest_reg_type]);
>           if (inst->bits1.ia1.dest_indirect_offset)
>               format (file, " %d", inst->bits1.ia1.dest_indirect_offset);
> -         string (file, "]");
> -         format (file, "<%d>", inst->bits1.ia1.dest_horiz_stride);
> +         string (file, "]<");
> +         err |= control (file, "horiz stride", horiz_stride, 
> inst->bits1.ia1.dest_horiz_stride, NULL);

missing the '<' printout?

Other than that,
Reviewed-by: Eric Anholt <e...@anholt.net>

Attachment: pgpqdoc_tCwvR.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to