summaryrefslogtreecommitdiffstats
path: root/src/mesa/shader
diff options
context:
space:
mode:
authorBrian Paul <[email protected]>2008-11-01 15:55:39 -0600
committerBrian Paul <[email protected]>2008-11-01 16:05:40 -0600
commitb625a0a47524de1b2968f9bf6a23ae0102726b5a (patch)
tree19b5d903598b355759c4a6c98c7fa3c7d5ed530a /src/mesa/shader
parent3d0d803313b624065e6549b1635d61e1162487ee (diff)
mesa: do scope replacement for while/for loops too
This fixes a function inlining bug involving vars declared inside loop bodies.
Diffstat (limited to 'src/mesa/shader')
-rw-r--r--src/mesa/shader/slang/slang_compile_operation.c10
1 files changed, 7 insertions, 3 deletions
diff --git a/src/mesa/shader/slang/slang_compile_operation.c b/src/mesa/shader/slang/slang_compile_operation.c
index 1002be16cb7..53cf6faff97 100644
--- a/src/mesa/shader/slang/slang_compile_operation.c
+++ b/src/mesa/shader/slang/slang_compile_operation.c
@@ -80,21 +80,23 @@ slang_replace_scope(slang_operation *oper,
slang_variable_scope *newScope)
{
GLuint i;
+
if (oper->locals != newScope &&
oper->locals->outer_scope == oldScope) {
+ /* found. replace old w/ new */
oper->locals->outer_scope = newScope;
}
if (oper->type == SLANG_OPER_VARIABLE_DECL) {
+ /* search/replace in the initializer */
slang_variable *var;
var = _slang_locate_variable(oper->locals, oper->a_id, GL_TRUE);
if (var && var->initializer) {
- printf("replace scope for %s initializer\n",
- (char *) var->a_name);
slang_replace_scope(var->initializer, oldScope, newScope);
}
}
+ /* search/replace in children */
for (i = 0; i < oper->num_children; i++) {
slang_replace_scope(&oper->children[i], oldScope, newScope);
}
@@ -159,7 +161,9 @@ slang_operation_copy(slang_operation * x, const slang_operation * y)
/* If this operation declares a new scope, we need to make sure
* all children point to it, not the original operation's scope!
*/
- if (x->type == SLANG_OPER_BLOCK_NEW_SCOPE) {
+ if (x->type == SLANG_OPER_BLOCK_NEW_SCOPE ||
+ x->type == SLANG_OPER_WHILE ||
+ x->type == SLANG_OPER_FOR) {
slang_replace_scope(x, y->locals, x->locals);
}