diff options
| author | Kenneth Graunke <kenneth@whitecape.org> | 2016-04-29 14:04:17 -0700 |
|---|---|---|
| committer | Kenneth Graunke <kenneth@whitecape.org> | 2016-05-15 23:59:10 -0700 |
| commit | 3bf27a9a002fd5c23937e4e34aff35c94824008f (patch) | |
| tree | 1c16414d5b916228b6b496d0c0400b5c71f5df82 /src/compiler/glsl/opt_constant_folding.cpp | |
| parent | 8e59670bcf925714896fb275a364e97a5137f0f7 (diff) | |
| download | external_mesa3d-3bf27a9a002fd5c23937e4e34aff35c94824008f.zip external_mesa3d-3bf27a9a002fd5c23937e4e34aff35c94824008f.tar.gz external_mesa3d-3bf27a9a002fd5c23937e4e34aff35c94824008f.tar.bz2 | |
glsl: Remove bonus tree walking in opt_constant_folding().
It looks like this was missed when converting opt_constant_folding()
from a hierarchical visitor to an rvalue visitor in 6606fde3.
ir_rvalue_visitor already processes values on the way back up the tree,
so we will have already visited every child node. There's no point in
doing it again.
No change in shader-db.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Diffstat (limited to 'src/compiler/glsl/opt_constant_folding.cpp')
| -rw-r--r-- | src/compiler/glsl/opt_constant_folding.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/compiler/glsl/opt_constant_folding.cpp b/src/compiler/glsl/opt_constant_folding.cpp index 150a17b..de13c9e 100644 --- a/src/compiler/glsl/opt_constant_folding.cpp +++ b/src/compiler/glsl/opt_constant_folding.cpp @@ -89,8 +89,6 @@ ir_constant_folding_visitor::handle_rvalue(ir_rvalue **rvalue) if (constant) { *rvalue = constant; this->progress = true; - } else { - (*rvalue)->accept(this); } } |