commit 5c0e023c4933cff33e40a8c413cba890a3e6271f
author Olli Etuaho <oetuaho@nvidia.com> Wed Nov 11 15:55:59 2015 +0200
committer Olli Etuaho <oetuaho@nvidia.com> Thu Nov 19 08:57:00 2015 +0000
tree 3ed940ab5bbd764882c397aba24589f5671216bd
parent 36b0514accbafba86c527af8ed1e8f7587fa7b15

Qualify stored constant union data with const

This prevents accidentally changing data that may be shared between
multiple TIntermConstantUnion nodes.

Besides making the code less prone to bugs in general, this will make it
easier to implement constant folding of array constructors.

BUG=541551
TEST=angle_unittests, WebGL conformance tests

Change-Id: I4f3059f70b841d9dd0cf20fea4d37684da9cd47e
Reviewed-on: https://chromium-review.googlesource.com/312440
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tryjob-Request: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Tested-by: Olli Etuaho <oetuaho@nvidia.com>

src/compiler/translator/Intermediate.cpp

diff --git a/src/compiler/translator/Intermediate.cpp b/src/compiler/translator/Intermediate.cpp
index c973535..0adb721 100644
--- a/src/compiler/translator/Intermediate.cpp
+++ b/src/compiler/translator/Intermediate.cpp
@@ -375,8 +375,9 @@
 // Returns the constant union node created.
 //
 
-TIntermConstantUnion *TIntermediate::addConstantUnion(
-    TConstantUnion *constantUnion, const TType &type, const TSourceLoc &line)
+TIntermConstantUnion *TIntermediate::addConstantUnion(const TConstantUnion *constantUnion,
+                                                      const TType &type,
+                                                      const TSourceLoc &line)
 {
     TIntermConstantUnion *node = new TIntermConstantUnion(constantUnion, type);
     node->setLine(line);