commit 6341939f3b7fc90a5360198bc6d564b09a6b4ab0
author Arun Patole <apatole@nvidia.com> Fri Mar 13 11:51:07 2015 +0530
committer Olli Etuaho <oetuaho@nvidia.com> Tue Mar 17 15:37:20 2015 +0000
tree c9f7ec0dbb8be09dcc0a6e91170e02d7ffad3af4
parent 0110e17264fb70ac5dcb72032d057d4051411a7c

Fix code duplication in OutputHLSL

While debugging, I came across the piece of code in
OutputHLSL::OutputHLSL which is unnecessarily duplicated,
this change just fixes it.

Change-Id: I0549d58696417ccf651bacfa19f489b290d4ba6e
Reviewed-on: https://chromium-review.googlesource.com/259960
Reviewed-by: Olli Etuaho <oetuaho@nvidia.com>
Tested-by: Olli Etuaho <oetuaho@nvidia.com>

src/compiler/translator/OutputHLSL.cpp

diff --git a/src/compiler/translator/OutputHLSL.cpp b/src/compiler/translator/OutputHLSL.cpp
index f3aa556..29ce37b 100644
--- a/src/compiler/translator/OutputHLSL.cpp
+++ b/src/compiler/translator/OutputHLSL.cpp
@@ -143,16 +143,10 @@
 
     if (mOutputType == SH_HLSL9_OUTPUT)
     {
-        if (mShaderType == GL_FRAGMENT_SHADER)
-        {
-            // Reserve registers for dx_DepthRange, dx_ViewCoords and dx_DepthFront
-            mUniformHLSL->reserveUniformRegisters(3);
-        }
-        else
-        {
-            // Reserve registers for dx_DepthRange, dx_ViewAdjust and dx_ViewCoords
-            mUniformHLSL->reserveUniformRegisters(3);
-        }
+        // Fragment shaders need dx_DepthRange, dx_ViewCoords and dx_DepthFront.
+        // Vertex shaders need a slightly different set: dx_DepthRange, dx_ViewCoords and dx_ViewAdjust.
+        // In both cases total 3 uniform registers need to be reserved.
+        mUniformHLSL->reserveUniformRegisters(3);
     }
 
     // Reserve registers for the default uniform block and driver constants