Mesa (master): glsl: minor simplification in assign_varying_locations()

Brian Paul brianp at kemper.freedesktop.org
Thu Dec 21 02:19:53 UTC 2017


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

Author: Brian Paul <brianp at vmware.com>
Date:   Fri Dec 15 14:25:20 2017 -0700

glsl: minor simplification in assign_varying_locations()

Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>

---

 src/compiler/glsl/link_varyings.cpp | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/compiler/glsl/link_varyings.cpp b/src/compiler/glsl/link_varyings.cpp
index 9039c3b8b8..6d74f9a463 100644
--- a/src/compiler/glsl/link_varyings.cpp
+++ b/src/compiler/glsl/link_varyings.cpp
@@ -2514,11 +2514,9 @@ assign_varying_locations(struct gl_context *ctx,
        */
       foreach_in_list(ir_instruction, node, consumer->ir) {
          ir_variable *const input_var = node->as_variable();
-
-         if (input_var == NULL || input_var->data.mode != ir_var_shader_in)
-            continue;
-
-         matches.record(NULL, input_var);
+         if (input_var && input_var->data.mode == ir_var_shader_in) {
+            matches.record(NULL, input_var);
+         }
       }
    }
 




More information about the mesa-commit mailing list