commit beb6dc74a2bcd35404a2054aae2927a193c4e240
author Olli Etuaho <oetuaho@nvidia.com> Thu Dec 14 16:03:03 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Thu Dec 21 19:46:57 2017 +0000
tree 80e9956916fed8b5d5e95b6ac174e243bd87d42d
parent 1bb8528c057f103070d8523c512adf1e0f83c2c3

Always use TFunction instead of TFunctionSymbolInfo

This reduces unnecessary memory allocations and conversions between
different objects containing the same data.

BUG=angleproject:2267
TEST=angle_unittests

Change-Id: I87316509ab1cd6d36756ff6af7fa2b5c5a76a8ea
Reviewed-on: https://chromium-review.googlesource.com/827134
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>

src/compiler/translator/OutputGLSLBase.cpp

diff --git a/src/compiler/translator/OutputGLSLBase.cpp b/src/compiler/translator/OutputGLSLBase.cpp
index 4ffa5b2..db5519e 100644
--- a/src/compiler/translator/OutputGLSLBase.cpp
+++ b/src/compiler/translator/OutputGLSLBase.cpp
@@ -915,7 +915,7 @@
     if (type.isArray())
         out << ArrayString(type);
 
-    out << " " << hashFunctionNameIfNeeded(*node->getFunctionSymbolInfo());
+    out << " " << hashFunctionNameIfNeeded(node->getFunction());
 
     out << "(";
     writeFunctionParameters(*(node->getSequence()));
@@ -1155,18 +1155,6 @@
     }
 }
 
-TString TOutputGLSLBase::hashFunctionNameIfNeeded(const TFunctionSymbolInfo &info)
-{
-    if (info.isMain())
-    {
-        return info.getName();
-    }
-    else
-    {
-        return hashName(info.getNameObj());
-    }
-}
-
 bool TOutputGLSLBase::structDeclared(const TStructure *structure) const
 {
     ASSERT(structure);