[Mesa-dev] [PATCH] anv: Do not assert() on VK_ATTACHMENT_UNUSED

[Nanley Chery]

On Mon, Oct 09, 2017 at 10:42:06PM +0200, Józef Kucia wrote:
> ---
>  src/intel/vulkan/genX_cmd_buffer.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 

This patch is
Reviewed-by: Nanley Chery <nanley.g.chery at intel.com>

> diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c
> index 93d360a1bb..43437c8eb0 100644
> --- a/src/intel/vulkan/genX_cmd_buffer.c
> +++ b/src/intel/vulkan/genX_cmd_buffer.c
> @@ -2996,10 +2996,11 @@ cmd_buffer_subpass_sync_fast_clear_values(struct anv_cmd_buffer *cmd_buffer)
>         * render pass and used in the subpass.
>         */
>        const uint32_t a = state->subpass->color_attachments[i].attachment;
> -      assert(a < state->pass->attachment_count);
>        if (a == VK_ATTACHMENT_UNUSED)
>           continue;
>  
> +      assert(a < state->pass->attachment_count);
> +
>        /* Store some information regarding this attachment. */
>        const struct anv_attachment_state *att_state = &state->attachments[a];
>        const struct anv_image_view *iview = state->framebuffer->attachments[a];
> -- 
> 2.13.6
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev