Mesa (master): swr/rast: move construction of const above goto

Tim Rowley torowley at kemper.freedesktop.org
Sat Apr 29 01:03:07 UTC 2017


Module: Mesa
Branch: master
Commit: 20f3a30219a8baba604815eba035a720c4c83b94
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=20f3a30219a8baba604815eba035a720c4c83b94

Author: Tim Rowley <timothy.o.rowley at intel.com>
Date:   Thu Apr 13 16:11:09 2017 -0500

swr/rast: move construction of const above goto

Fixes gcc error for SIMD16 FE.

Reviewed-by: Bruce Cherniak <bruce.cherniak at intel.com>

---

 src/gallium/drivers/swr/rasterizer/core/binner.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/gallium/drivers/swr/rasterizer/core/binner.cpp b/src/gallium/drivers/swr/rasterizer/core/binner.cpp
index 3c94004c31..958f0a2644 100644
--- a/src/gallium/drivers/swr/rasterizer/core/binner.cpp
+++ b/src/gallium/drivers/swr/rasterizer/core/binner.cpp
@@ -2499,6 +2499,8 @@ void BinPostSetupLines_simd16(
         primMask = primMask & ~maskOutsideScissor;
     }
 
+    const simdscalar unused = _simd_setzero_ps();
+
     if (!primMask)
     {
         goto endBinLines;
@@ -2524,8 +2526,6 @@ void BinPostSetupLines_simd16(
     __m128 vHorizZ[2][KNOB_SIMD_WIDTH]; // KNOB_SIMD16_WIDTH
     __m128 vHorizW[2][KNOB_SIMD_WIDTH]; // KNOB_SIMD16_WIDTH
 
-    const simdscalar unused = _simd_setzero_ps();
-
     vTranspose3x8(vHorizX[0], _simd16_extract_ps(prim[0].x, 0), _simd16_extract_ps(prim[1].x, 0), unused);
     vTranspose3x8(vHorizY[0], _simd16_extract_ps(prim[0].y, 0), _simd16_extract_ps(prim[1].y, 0), unused);
     vTranspose3x8(vHorizZ[0], _simd16_extract_ps(prim[0].z, 0), _simd16_extract_ps(prim[1].z, 0), unused);




More information about the mesa-commit mailing list