turnip: move trace_start_gmem_store before cond exec
authorChia-I Wu <olvaffe@gmail.com>
Thu, 25 Aug 2022 15:47:50 +0000 (08:47 -0700)
committerMarge Bot <emma+marge@anholt.net>
Thu, 25 Aug 2022 21:00:14 +0000 (21:00 +0000)
Suggested by Danylo.

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

src/freedreno/vulkan/tu_clear_blit.c

index 56e519d..bb96557 100644 (file)
@@ -3323,6 +3323,8 @@ tu_store_gmem_attachment(struct tu_cmd_buffer *cmd,
    if (!dst->store && !dst->store_stencil)
       return;
 
+   trace_start_gmem_store(&cmd->trace, cs);
+
    /* Unconditional store should happen only if attachment was cleared,
     * which could have happened either by load_op or via vkCmdClearAttachments.
     */
@@ -3351,8 +3353,6 @@ tu_store_gmem_attachment(struct tu_cmd_buffer *cmd,
    bool store_common = dst->store && !resolve_d32s8_s8;
    bool store_separate_stencil = dst->store_stencil || resolve_d32s8_s8;
 
-   trace_start_gmem_store(&cmd->trace, cs);
-
    /* use fast path when render area is aligned, except for unsupported resolve cases */
    if (!unaligned && !resolve_d24s8_s8 &&
        (a == gmem_a || blit_can_resolve(dst->format))) {