We don't need to keep the param variable around for function prototypes.

Saves 20% memory for global symbol initialization R=alokp@chromium.org Review URL: https://codereview.appspot.com/9152043 git-svn-id: https://angleproject.googlecode.com/svn/trunk@2196 736b8ea6-26fd-11df-bfd4-992fa37f6226

We don't need to keep the param variable around for function prototypes.
fd35e3dd · alokp@chromium.org · e0b8416a · fd35e3dd · fd35e3dd
Commit fd35e3dd authored May 06, 2013 by alokp@chromium.org
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

glslang.y src/compiler/glslang.y +2 -2

glslang_tab.cpp src/compiler/glslang_tab.cpp +2 -2

No files found.
--- a/src/compiler/glslang.y
+++ b/src/compiler/glslang.y
@@ -967,9 +967,9 @@ declaration
            const TParameter &param = function.getParam(i);
            if (param.name != 0)
            {
-                TVariable *variable = new TVariable(param.name, *param.type);
+                TVariable variable(param.name, *param.type);
-                prototype = context->intermediate.growAggregate(prototype, context->intermediate.addSymbol(variable->getUniqueId(), variable->getName(), variable->getType(), $1.line), $1.line);
+                prototype = context->intermediate.growAggregate(prototype, context->intermediate.addSymbol(variable.getUniqueId(), variable.getName(), variable.getType(), $1.line), $1.line);
            }
            else
            {

--- a/src/compiler/glslang_tab.cpp
+++ b/src/compiler/glslang_tab.cpp
@@ -3085,9 +3085,9 @@ yyreduce:
            const TParameter &param = function.getParam(i);
            if (param.name != 0)
            {
-                TVariable *variable = new TVariable(param.name, *param.type);
+                TVariable variable(param.name, *param.type);
-                prototype = context->intermediate.growAggregate(prototype, context->intermediate.addSymbol(variable->getUniqueId(), variable->getName(), variable->getType(), (yyvsp[(1) - (2)].interm).line), (yyvsp[(1) - (2)].interm).line);
+                prototype = context->intermediate.growAggregate(prototype, context->intermediate.addSymbol(variable.getUniqueId(), variable.getName(), variable.getType(), (yyvsp[(1) - (2)].interm).line), (yyvsp[(1) - (2)].interm).line);
            }
            else
            {