anv: Fix warning 3DSTATE_VERTEX_ELEMENTS setup
authorKristian Høgsberg Kristensen <kristian.h.kristensen@intel.com>
Sun, 14 Feb 2016 22:17:08 +0000 (14:17 -0800)
committerKristian Høgsberg Kristensen <kristian.h.kristensen@intel.com>
Tue, 16 Feb 2016 01:32:07 +0000 (17:32 -0800)
This is a little more subtle. If elem_count is 0, nothing else happens
in this function, so we return early to avoid warning about
uninitialized 'p'.

src/vulkan/genX_pipeline_util.h

index 077281d2f565fa14f2a79e65238d38102e3ea8e1..696e2be7c3f08cc3c4db89f3285e02fed4cb4125 100644 (file)
@@ -85,14 +85,15 @@ emit_vertex_input(struct anv_pipeline *pipeline,
 #endif
 
    uint32_t elem_count = __builtin_popcount(elements) + needs_svgs_elem;
+   if (elem_count == 0)
+      return;
 
    uint32_t *p;
-   if (elem_count > 0) {
-      const uint32_t num_dwords = 1 + elem_count * 2;
-      p = anv_batch_emitn(&pipeline->batch, num_dwords,
-                          GENX(3DSTATE_VERTEX_ELEMENTS));
-      memset(p + 1, 0, (num_dwords - 1) * 4);
-   }
+
+   const uint32_t num_dwords = 1 + elem_count * 2;
+   p = anv_batch_emitn(&pipeline->batch, num_dwords,
+                       GENX(3DSTATE_VERTEX_ELEMENTS));
+   memset(p + 1, 0, (num_dwords - 1) * 4);
 
    for (uint32_t i = 0; i < info->vertexAttributeDescriptionCount; i++) {
       const VkVertexInputAttributeDescription *desc =