Local variables hide function arguments

Definitions of variables inside functions should be located in a scope different from function's arguments scope. Change-Id: I89fe1f527681c68963f4a69c71b78975787a410c Reviewed-on: https://swiftshader-review.googlesource.com/4180Reviewed-by: Nicolas Capens <capn@google.com> Tested-by: Veranika Liaukevich <veranika@google.com>

Local variables hide function arguments
1aa4b61c · Veranika Liaukevich · eced26a1 · 1aa4b61c · 1aa4b61c
Commit 1aa4b61c authored Oct 28, 2015 by Veranika Liaukevich
Expand all Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

glslang.y src/OpenGL/compiler/glslang.y +1 -1

glslang_tab.cpp src/OpenGL/compiler/glslang_tab.cpp +0 -0

No files found.
--- a/src/OpenGL/compiler/glslang.y
+++ b/src/OpenGL/compiler/glslang.y
@@ -1718,7 +1718,7 @@ function_definition
        $1.intermAggregate = paramNodes;
        context->setLoopNestingLevel(0);
    }
-    compound_statement_no_new_scope {
+    compound_statement {
        //?? Check that all paths return a value if return type != void ?
        //   May be best done as post process phase on intermediate code
        if (context->getCurrentFunctionType()->getBasicType() != EbtVoid && ! context->getFunctionReturnsValue()) {

--- a/src/OpenGL/compiler/glslang_tab.cpp
+++ b/src/OpenGL/compiler/glslang_tab.cpp