[Mesa-dev] [PATCH] radv: drop assert on bindingDescriptorCount > 0

Dave Airlie airlied at gmail.com
Thu Mar 8 00:16:23 UTC 2018


From: Dave Airlie <airlied at redhat.com>

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 <airlied at redhat.com>
---
 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;
-- 
2.14.3



More information about the mesa-dev mailing list