commit ea39a2239f2366c88efd3e01b97a5caa83179773
author Olli Etuaho <oetuaho@nvidia.com> Thu Jul 06 12:47:59 2017 +0300
committer Commit Bot <commit-bot@chromium.org> Sun Jul 09 07:14:10 2017 +0000
tree 29077212404963bbc5bc69a58846310ef4e705c1
parent 145e3cbbbb075e32874811ec9656c89218af90d0

Simplify queueReplacement AST transform helper

queueReplacement is always called to replace the node that's currently
being visited in the traverser. The currently visited node can be
fetched automatically from the traversal path so it can be removed
from parameters of queueReplacement.

BUG=angleproject:2100
TEST=angle_unittests

Change-Id: I62ab6d1cd9c0d2b4c260af9f7c85bc156fb3f349
Reviewed-on: https://chromium-review.googlesource.com/562336
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>

src/compiler/translator/RemoveDynamicIndexing.cpp

diff --git a/src/compiler/translator/RemoveDynamicIndexing.cpp b/src/compiler/translator/RemoveDynamicIndexing.cpp
index 5e9abe0..02b1112 100644
--- a/src/compiler/translator/RemoveDynamicIndexing.cpp
+++ b/src/compiler/translator/RemoveDynamicIndexing.cpp
@@ -493,7 +493,7 @@
                     node, tempIndex, createTempSymbol(fieldType), indexedWriteFunctionId);
                 insertionsAfter.push_back(indexedWriteCall);
                 insertStatementsInParentBlock(insertionsBefore, insertionsAfter);
-                queueReplacement(node, createTempSymbol(fieldType), OriginalNode::IS_DROPPED);
+                queueReplacement(createTempSymbol(fieldType), OriginalNode::IS_DROPPED);
                 mUsedTreeInsertion = true;
             }
             else
@@ -506,7 +506,7 @@
                 ASSERT(!mRemoveIndexSideEffectsInSubtree);
                 TIntermAggregate *indexingCall = CreateIndexFunctionCall(
                     node, EnsureSignedInt(node->getRight()), indexingFunctionId);
-                queueReplacement(node, indexingCall, OriginalNode::IS_DROPPED);
+                queueReplacement(indexingCall, OriginalNode::IS_DROPPED);
             }
         }
     }