glsl: Make mode_string function globally available
authorIan Romanick <ian.d.romanick@intel.com>
Wed, 25 Sep 2013 18:44:41 +0000 (11:44 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Wed, 30 Oct 2013 20:49:30 +0000 (13:49 -0700)
commitf8c579dc0fc0ffed41a86daecbcfe6c3b9843e61
tree1fc49c33a09dacca38a525ae0515a193797ba23b
parent2cb760d9945b30bed027a14c2a9ac20d07d7c5ba
glsl: Make mode_string function globally available

I made this a function (instead of a method of ir_variable) because it
made the change set smaller, and I expect that there will be an overload
that takes an ir_var_mode enum.  Having both functions used the same way
seemed better.

v2: Add missing case for ir_var_system_value.

v3: Change the ir_var_mode_count case to just break.  Move the assertion
and the return outside the switch-statment.  In the unlikely event that
var->mode is an invalid value other than ir_var_mode_count, the
assertion will still fire, and in release builds we won't wind up
returning a garbage pointer.  Suggested by Paul.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Paul Berry <stereotype441@gmail.com>
src/glsl/ir.cpp
src/glsl/ir.h
src/glsl/linker.cpp