diff options
author | Jason Ekstrand <jason.ekstrand@intel.com> | 2016-06-06 09:15:03 -0700 |
---|---|---|
committer | Jason Ekstrand <jason.ekstrand@intel.com> | 2016-06-10 09:43:03 -0700 |
commit | c13c5ac561f6475d08c35d2a88a829e6ce36e98c (patch) | |
tree | 38db16f5939ad77d15dd0c00a0b68b907f9cc60e /src/intel/vulkan/anv_descriptor_set.c | |
parent | e2265926f25235f9be833984a5e365889a70ea74 (diff) | |
download | external_mesa3d-c13c5ac561f6475d08c35d2a88a829e6ce36e98c.zip external_mesa3d-c13c5ac561f6475d08c35d2a88a829e6ce36e98c.tar.gz external_mesa3d-c13c5ac561f6475d08c35d2a88a829e6ce36e98c.tar.bz2 |
anv/descriptor_set: Ensure that bindings are always in increasing order
Since applications are allowed to specify some set of bindings which need
not be dense they also need not be in order. For most things, this doesn't
matter, but it could result getting the wrong dynamic offsets. This adds a
quick-and-dirty sort to ensure that everything is always in increasing
order of binding index.
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Cc: "12.0" <mesa-stable@lists.freedesktop.org>
Diffstat (limited to 'src/intel/vulkan/anv_descriptor_set.c')
-rw-r--r-- | src/intel/vulkan/anv_descriptor_set.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/src/intel/vulkan/anv_descriptor_set.c b/src/intel/vulkan/anv_descriptor_set.c index c977318..448ae0e 100644 --- a/src/intel/vulkan/anv_descriptor_set.c +++ b/src/intel/vulkan/anv_descriptor_set.c @@ -89,6 +89,19 @@ VkResult anv_CreateDescriptorSetLayout( for (uint32_t j = 0; j < pCreateInfo->bindingCount; j++) { const VkDescriptorSetLayoutBinding *binding = &pCreateInfo->pBindings[j]; uint32_t b = binding->binding; + /* We temporarily store the pointer to the binding in the + * immutable_samplers pointer. This provides us with a quick-and-dirty + * way to sort the bindings by binding number. + */ + set_layout->binding[b].immutable_samplers = (void *)binding; + } + + for (uint32_t b = 0; b <= max_binding; b++) { + const VkDescriptorSetLayoutBinding *binding = + (void *)set_layout->binding[b].immutable_samplers; + + if (binding == NULL) + continue; assert(binding->descriptorCount > 0); #ifndef NDEBUG |