commit 57ea533f79a7a30224ea641785cbd7a9485d33ed
author Kai Ninomiya <kainino@chromium.org> Wed Nov 22 14:04:48 2017 -0800
committer Commit Bot <commit-bot@chromium.org> Tue Nov 28 23:33:41 2017 +0000
tree 26f25d4e8717624115203ab6226d1673c02244b0
parent 5201ce127afea0ef1d34e161ead085fe50e73da4

TType: Store array sizes vector as a pointer.

This makes TType a literal type, and thus is something that could be
constexpr.

Work started by jmadill here: https://crrev.com/c/776278

Bug: angleproject:1432
Change-Id: I707ddf81eaf029f49d62d2836b7166d265cbdfa1
Reviewed-on: https://chromium-review.googlesource.com/786316
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

src/compiler/translator/OutputHLSL.cpp

diff --git a/src/compiler/translator/OutputHLSL.cpp b/src/compiler/translator/OutputHLSL.cpp
index bb18beb..d5ff761 100644
--- a/src/compiler/translator/OutputHLSL.cpp
+++ b/src/compiler/translator/OutputHLSL.cpp
@@ -38,9 +38,12 @@
 {
     TStringStream fnName;
     fnName << prefix << "_";
-    for (unsigned int arraySize : type.getArraySizes())
+    if (type.isArray())
     {
-        fnName << arraySize << "_";
+        for (unsigned int arraySize : *type.getArraySizes())
+        {
+            fnName << arraySize << "_";
+        }
     }
     fnName << TypeString(type);
     return fnName.str();