Yeah, noticed that CTS fail this morning too. :)

Reviewed-by: Samuel Pitoiset <[email protected]>

On 03/08/2018 01:16 AM, Dave Airlie wrote:
From: Dave Airlie <[email protected]>

The spec is pretty clear that this can be 0, and that it operates
as a reserved binding.

Fixes:
dEQP-VK.binding_model.descriptor_update.empty_descriptor.uniform_buffer

Signed-off-by: Dave Airlie <[email protected]>
---
  src/amd/vulkan/radv_descriptor_set.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/src/amd/vulkan/radv_descriptor_set.c 
b/src/amd/vulkan/radv_descriptor_set.c
index daff7b2fcd..3d56f8c217 100644
--- a/src/amd/vulkan/radv_descriptor_set.c
+++ b/src/amd/vulkan/radv_descriptor_set.c
@@ -137,7 +137,6 @@ VkResult radv_CreateDescriptorSetLayout(
                }
set_layout->size = align(set_layout->size, alignment);
-               assert(binding->descriptorCount > 0);
                set_layout->binding[b].type = binding->descriptorType;
                set_layout->binding[b].array_size = binding->descriptorCount;
                set_layout->binding[b].offset = set_layout->size;

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to