summaryrefslogtreecommitdiffstats
path: root/src/glsl/lower_packed_varyings.cpp
diff options
context:
space:
mode:
authorIan Romanick <ian.d.romanick@intel.com>2014-07-08 18:53:09 -0700
committerIan Romanick <ian.d.romanick@intel.com>2014-09-30 13:34:42 -0700
commita99482482d74ba654d8ec15d0a09e5b3cb0160e9 (patch)
treed4723379eb3ae1fd4472c558031b2e302a15d7a9 /src/glsl/lower_packed_varyings.cpp
parent7625babfae6c5e86ab349c1a081816fbbcc48d17 (diff)
downloadexternal_mesa3d-a99482482d74ba654d8ec15d0a09e5b3cb0160e9.zip
external_mesa3d-a99482482d74ba654d8ec15d0a09e5b3cb0160e9.tar.gz
external_mesa3d-a99482482d74ba654d8ec15d0a09e5b3cb0160e9.tar.bz2
glsl: Never put ir_var_temporary variables in the symbol table
Later patches will give every ir_var_temporary the same name in release builds. Adding a bunch of variables named "compiler_temp" to the symbol table can only cause problems. No change Valgrind massif results for a trimmed apitrace of dota2. Signed-off-by: Ian Romanick <ian.d.romanick@intel.com> Reviewed-by: Matt Turner <mattst88@gmail.com>
Diffstat (limited to 'src/glsl/lower_packed_varyings.cpp')
-rw-r--r--src/glsl/lower_packed_varyings.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/glsl/lower_packed_varyings.cpp b/src/glsl/lower_packed_varyings.cpp
index 7801483..5e844c7 100644
--- a/src/glsl/lower_packed_varyings.cpp
+++ b/src/glsl/lower_packed_varyings.cpp
@@ -261,6 +261,7 @@ lower_packed_varyings_visitor::run(exec_list *instructions)
!var->type->contains_integer());
/* Change the old varying into an ordinary global. */
+ assert(var->data.mode != ir_var_temporary);
var->data.mode = ir_var_auto;
/* Create a reference to the old varying. */