[Mesa-dev] [PATCH] i965: Don't hardcode FS in "validation failed!" message.

Kristian Høgsberg krh at bitplanet.net
Tue Oct 13 15:59:40 PDT 2015


On Tue, Oct 13, 2015 at 3:17 PM, Kenneth Graunke <kenneth at whitecape.org> wrote:
> Instead, print "Scalar VS" or "Scalar FS".  Otherwise it's really
> confusing which stage is broken.
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>

Reviewed-by: Kristian Høgsberg <krh at bitplanet.net>

> ---
>  src/mesa/drivers/dri/i965/brw_fs_validate.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_validate.cpp b/src/mesa/drivers/dri/i965/brw_fs_validate.cpp
> index d0e04f3..814c551 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_validate.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_validate.cpp
> @@ -32,7 +32,7 @@
>
>  #define fsv_assert(cond) \
>     if (!(cond)) { \
> -      fprintf(stderr, "ASSERT: FS validation failed!\n"); \
> +      fprintf(stderr, "ASSERT: Scalar %s validation failed!\n", stage_abbrev); \
>        dump_instruction(inst, stderr); \
>        fprintf(stderr, "%s:%d: %s\n", __FILE__, __LINE__, #cond); \
>        abort(); \
> --
> 2.6.1
>
> _______________________________________________
> 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