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>
diff --git a/src/compiler/translator/Compiler.cpp b/src/compiler/translator/Compiler.cpp
index 4ae8d7e..730f0c5 100644
--- a/src/compiler/translator/Compiler.cpp
+++ b/src/compiler/translator/Compiler.cpp
@@ -993,22 +993,22 @@
const TIntermFunctionPrototype *asFunctionPrototype = node->getAsFunctionPrototypeNode();
const TIntermFunctionDefinition *asFunctionDefinition = node->getAsFunctionDefinition();
- const TFunctionSymbolInfo *functionInfo = nullptr;
+ const TFunction *func = nullptr;
if (asFunctionDefinition)
{
- functionInfo = asFunctionDefinition->getFunctionSymbolInfo();
+ func = asFunctionDefinition->getFunction();
}
else if (asFunctionPrototype)
{
- functionInfo = asFunctionPrototype->getFunctionSymbolInfo();
+ func = asFunctionPrototype->getFunction();
}
- if (functionInfo == nullptr)
+ if (func == nullptr)
{
return false;
}
- size_t callDagIndex = mCallDag->findIndex(functionInfo->getId());
+ size_t callDagIndex = mCallDag->findIndex(func->uniqueId());
if (callDagIndex == CallDAG::InvalidIndex)
{
// This happens only for unimplemented prototypes which are thus unused