Reviewed-by: Bas Nieuwenhuizen
for the series.
On Tue, Jul 10, 2018 at 4:13 PM, Samuel Pitoiset
wrote:
> Now that 'set' can't be NULL because the meta operations no
> longer bind a NULL descriptor, the logic can be simplified
> a little bit.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_cmd_buffer.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/src/amd/vulkan/radv_cmd_buffer.c
> b/src/amd/vulkan/radv_cmd_buffer.c
> index 11f4c29db4..e85a9d7c98 100644
> --- a/src/amd/vulkan/radv_cmd_buffer.c
> +++ b/src/amd/vulkan/radv_cmd_buffer.c
> @@ -518,10 +518,8 @@ void radv_set_descriptor_set(struct radv_cmd_buffer
> *cmd_buffer,
> radv_get_descriptors_state(cmd_buffer, bind_point);
>
> descriptors_state->sets[idx] = set;
> - if (set)
> - descriptors_state->valid |= (1u << idx);
> - else
> - descriptors_state->valid &= ~(1u << idx);
> +
> + descriptors_state->valid |= (1u << idx); /* active descriptors */
> descriptors_state->dirty |= (1u << idx);
> }
>
> --
> 2.18.0
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev