[Mesa-dev] [PATCH 23/23] i965/disasm: Fix INTEL_DEBUG=fs on Broadwell for ARB_fp applications.

[Kristian Høgsberg]

On Sat, Jun 28, 2014 at 09:34:02PM -0700, Kenneth Graunke wrote:
> Apparently INTEL_DEBUG=fs has crashed on Broadwell for anything using
> ARB_fragment_program since commit 9cee3ff5.  We need to NULL-check the
> right field.
> 
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> Cc: "10.2" <mesa-stable at lists.freedesktop.org>

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

> ---
>  src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> I could've sworn I fixed this, but apparently I was thinking about
> commit 2f97119950515c841bca98a890e5110206bad945, which fixed a similar
> bug in the VS visitor.
> 
> diff --git a/src/mesa/drivers/dri/i965/gen8_fs_generator.cpp b/src/mesa/drivers/dri/i965/gen8_fs_generator.cpp
> index 4725774..ac8bf1d 100644
> --- a/src/mesa/drivers/dri/i965/gen8_fs_generator.cpp
> +++ b/src/mesa/drivers/dri/i965/gen8_fs_generator.cpp
> @@ -1255,7 +1255,7 @@ gen8_fs_generator::generate_code(exec_list *instructions)
>     int before_size = next_inst_offset - start_offset;
>  
>     if (unlikely(INTEL_DEBUG & DEBUG_WM)) {
> -      if (prog) {
> +      if (shader_prog) {
>           fprintf(stderr,
>                   "Native code for %s fragment shader %d (SIMD%d dispatch):\n",
>                  shader_prog->Label ? shader_prog->Label : "unnamed",
> -- 
> 2.0.0
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev