[Mesa-dev] [PATCH 01/19] radv: skip the decompression color pass when CMASK/FMASK/DCC are disabled

[Dave Airlie]

On 1 Oct. 2017 22:03, "Bas Nieuwenhuizen" <bas at basnieuwenhuizen.nl> wrote:

Why add this? It sounds like extra code for no reason?


Also why add it after the meta saves?

Dave.


On Fri, Sep 29, 2017 at 5:48 PM, Samuel Pitoiset
<samuel.pitoiset at gmail.com> wrote:
> This should be a no-op.
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/amd/vulkan/radv_meta_fast_clear.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/amd/vulkan/radv_meta_fast_clear.c
b/src/amd/vulkan/radv_meta_fast_clear.c
> index b92fb7cd5a..7e0738f44f 100644
> --- a/src/amd/vulkan/radv_meta_fast_clear.c
> +++ b/src/amd/vulkan/radv_meta_fast_clear.c
> @@ -365,6 +365,9 @@ radv_fast_clear_flush_image_inplace(struct
radv_cmd_buffer *cmd_buffer,
>         radv_meta_save_pass(&saved_pass_state, cmd_buffer);
>         radv_meta_save_graphics_reset_vport_scissor_novertex(&saved_state,
cmd_buffer);
>
> +       if (!image->cmask.size && !image->fmask.size &&
!image->dcc_offset)
> +               return;
> +
>         if (image->surface.dcc_size) {
>                 radv_emit_set_predication_state_from_image(cmd_buffer,
image, true);
>                 cmd_buffer->state.predicating = true;
> --
> 2.14.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171002/3b0fe377/attachment.html>