v3dv: drop incorrect assertion
authorIago Toral Quiroga <itoral@igalia.com>
Wed, 11 Mar 2020 07:37:48 +0000 (08:37 +0100)
committerMarge Bot <eric+marge@anholt.net>
Tue, 13 Oct 2020 21:21:28 +0000 (21:21 +0000)
It is possible to specify a depth/stencil clear but then have no
actual depth/stencil attachment used in the subpass. In that case
we are already skipping the store.

Fixes:
dEQP-VK.renderpass.suballocation.unused_clear_attachments.*depthonly_unused

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6766>

src/broadcom/vulkan/v3dv_meta_clear.c

index 9f736c8..e1f7baa 100644 (file)
@@ -370,8 +370,6 @@ emit_tlb_clear_job(struct v3dv_cmd_buffer *cmd_buffer,
    }
 
    cl_emit(rcl, TILE_RENDERING_MODE_CFG_ZS_CLEAR_VALUES, clear) {
-      assert(!ds_clear_value ||
-             subpass->ds_attachment.attachment != VK_ATTACHMENT_UNUSED);
       clear.z_clear_value = ds_clear_value ? ds_clear_value->depth : 1.0f;
       clear.stencil_clear_value = ds_clear_value ? ds_clear_value->stencil : 0;
    };