diff options
| author | Ian Romanick <[email protected]> | 2014-07-08 18:53:09 -0700 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2014-09-30 13:34:42 -0700 |
| commit | a99482482d74ba654d8ec15d0a09e5b3cb0160e9 (patch) | |
| tree | d4723379eb3ae1fd4472c558031b2e302a15d7a9 /src/glsl/lower_packed_varyings.cpp | |
| parent | 7625babfae6c5e86ab349c1a081816fbbcc48d17 (diff) | |
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 <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/glsl/lower_packed_varyings.cpp')
| -rw-r--r-- | src/glsl/lower_packed_varyings.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/glsl/lower_packed_varyings.cpp b/src/glsl/lower_packed_varyings.cpp index 78014831566..5e844c792e8 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. */ |