tree 65a56dd6039c5860b7bdb731f7567b26864225a0
parent 76e6565e23d8f01146a31a75b8acf7ff30b638c6
author Olli Etuaho <oetuaho@nvidia.com> 1490088654 +0000
committer Commit Bot <commit-bot@chromium.org> 1490868995 +0000

Prefer identifying functions by using symbol ids

The shader translator code is now structured in a way that ensures
that all function definition, function prototype and function call
nodes store the integer symbol id for the function. This is guaranteed
regardless of whether the function node is added while parsing or as a
result of an AST transformation. TIntermAggregate nodes, which include
function calls and constructors can now only be created by calling one
of the TIntermAggregate::Create*() functions to ensure they have all
the necessary properties.

This makes it possible to keep track of functions using integer ids
instead of their mangled name strings when generating the call graph
and when using TLValueTrackingTraverser.

This commit includes a few other small cleanups to the CallDAG class
as well.

BUG=angleproject:1490
TEST=angle_unittests, angle_end2end_tests

Change-Id: Idd1013506cbe4c3380e20d90524a9cd09b890259
Reviewed-on: https://chromium-review.googlesource.com/459603
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>