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/OutputTree.cpp

diff --git a/src/compiler/translator/OutputTree.cpp b/src/compiler/translator/OutputTree.cpp
index 8bacce9..67efc0f 100644
--- a/src/compiler/translator/OutputTree.cpp
+++ b/src/compiler/translator/OutputTree.cpp
@@ -13,13 +13,6 @@
 namespace
 {
 
-void OutputFunction(TInfoSinkBase &out, const char *str, const TFunctionSymbolInfo *info)
-{
-    const char *internal = info->getNameObj().isInternal() ? " (internal function)" : "";
-    out << str << internal << ": " << info->getNameObj().getString() << " (symbol id "
-        << info->getId().get() << ")";
-}
-
 void OutputFunction(TInfoSinkBase &out, const char *str, const TFunction *func)
 {
     const char *internal =
@@ -364,7 +357,7 @@
 bool TOutputTraverser::visitFunctionPrototype(Visit visit, TIntermFunctionPrototype *node)
 {
     OutputTreeText(mOut, node, mDepth);
-    OutputFunction(mOut, "Function Prototype", node->getFunctionSymbolInfo());
+    OutputFunction(mOut, "Function Prototype", node->getFunction());
     mOut << " (" << node->getCompleteString() << ")";
     mOut << "\n";