Mesa (master): glsl: Assert that inputs have a location assigned

Ian Romanick idr at kemper.freedesktop.org
Wed May 18 17:54:09 UTC 2016


Module: Mesa
Branch: master
Commit: 2ef4b5bc932bac100570d377551959c5655e37a3
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2ef4b5bc932bac100570d377551959c5655e37a3

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Mon May 16 18:01:10 2016 -0700

glsl: Assert that inputs have a location assigned

This catches a problem previously undetected until deep in the backend.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>

---

 src/compiler/glsl/ir_set_program_inouts.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/compiler/glsl/ir_set_program_inouts.cpp b/src/compiler/glsl/ir_set_program_inouts.cpp
index 6768d82..183b13b 100644
--- a/src/compiler/glsl/ir_set_program_inouts.cpp
+++ b/src/compiler/glsl/ir_set_program_inouts.cpp
@@ -94,6 +94,8 @@ mark(struct gl_program *prog, ir_variable *var, int offset, int len,
     */
 
    for (int i = 0; i < len; i++) {
+      assert(var->data.location != -1);
+
       int idx = var->data.location + var->data.index + offset + i;
       bool is_patch_generic = var->data.patch &&
                               idx != VARYING_SLOT_TESS_LEVEL_INNER &&




More information about the mesa-commit mailing list