From: Connor Abbott <connor.w.abbott@intel.com>
Date: Thu, 16 Jul 2015 18:04:22 +0000 (-0700)
Subject: nir/spirv: don't declare builtin blocks
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=b2cfd8506034e733f71722334330b5df0bf94ef8;p=mesa.git

nir/spirv: don't declare builtin blocks

They aren't used, and the backend was barfing on them. Also, remove a
hack in in compiler.cpp now that they're gone.
---

diff --git a/src/glsl/nir/spirv_to_nir.c b/src/glsl/nir/spirv_to_nir.c
index ee44a3f291e..65a995c29de 100644
--- a/src/glsl/nir/spirv_to_nir.c
+++ b/src/glsl/nir/spirv_to_nir.c
@@ -359,6 +359,7 @@ struct_member_decoration_cb(struct vtn_builder *b,
                                                  ctx->type->members[member]);
       ctx->type->members[member]->is_builtin = true;
       ctx->type->members[member]->builtin = dec->literals[0];
+      ctx->type->builtin_block = true;
       break;
    case SpvDecorationOffset:
       ctx->type->offsets[member] = dec->literals[0];
@@ -404,7 +405,7 @@ vtn_handle_type(struct vtn_builder *b, SpvOp opcode,
 {
    struct vtn_value *val = vtn_push_value(b, w[1], vtn_value_type_type);
 
-   val->type = ralloc(b, struct vtn_type);
+   val->type = rzalloc(b, struct vtn_type);
    val->type->is_builtin = false;
 
    switch (opcode) {
@@ -1230,13 +1231,18 @@ vtn_handle_variables(struct vtn_builder *b, SpvOp opcode,
       var->type = type->type;
       var->name = ralloc_strdup(var, val->name);
 
-      if (type->block)
+      bool builtin_block = false;
+      if (type->block) {
          var->interface_type = type->type;
-      else if (glsl_type_is_array(type->type) &&
-               (type->array_element->block || type->array_element->buffer_block))
+         builtin_block = type->builtin_block;
+      } else if (glsl_type_is_array(type->type) &&
+                 (type->array_element->block ||
+                  type->array_element->buffer_block)) {
          var->interface_type = type->array_element->type;
-      else
+         builtin_block = type->array_element->builtin_block;
+      } else {
          var->interface_type = NULL;
+      }
 
       switch ((SpvStorageClass)w[3]) {
       case SpvStorageClassUniform:
@@ -1294,6 +1300,12 @@ vtn_handle_variables(struct vtn_builder *b, SpvOp opcode,
       if (var->data.mode == nir_var_uniform && var->interface_type)
          break;
 
+      /* Builtin blocks are lowered to individual variables during SPIR-V ->
+       * NIR, so don't declare them either.
+       */
+      if (builtin_block)
+         break;
+
       switch (var->data.mode) {
       case nir_var_shader_in:
          exec_list_push_tail(&b->shader->inputs, &var->node);
diff --git a/src/glsl/nir/spirv_to_nir_private.h b/src/glsl/nir/spirv_to_nir_private.h
index fa0489b877d..a964cc80fad 100644
--- a/src/glsl/nir/spirv_to_nir_private.h
+++ b/src/glsl/nir/spirv_to_nir_private.h
@@ -96,6 +96,11 @@ struct vtn_type {
    /* for structs, whether it was decorated as an "SSBO-like" block */
    bool buffer_block;
 
+   /* for structs with block == true, whether this is a builtin block (i.e. a
+    * block that contains only builtins).
+    */
+   bool builtin_block;
+
    /* for arrays and matrices, the array stride */
    unsigned stride;
 
diff --git a/src/vulkan/compiler.cpp b/src/vulkan/compiler.cpp
index cf34e7b4414..9152de63ec9 100644
--- a/src/vulkan/compiler.cpp
+++ b/src/vulkan/compiler.cpp
@@ -959,10 +959,6 @@ setup_nir_io(struct gl_program *prog,
    }
 
    foreach_list_typed(nir_variable, var, node, &shader->outputs) {
-      /* XXX glslang gives us this but we never use it */
-      if (!strcmp(var->name, "gl_PerVertex"))
-         continue;
-
       prog->OutputsWritten |= BITFIELD64_BIT(var->data.location);
    }
 }