Mesa (master): i965: Separate gen < 8 and gen >= 8 paths explicitly in wrap_mode()

Topi Pohjolainen tpohjola at kemper.freedesktop.org
Tue Jun 27 07:47:27 UTC 2017


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

Author: Topi Pohjolainen <topi.pohjolainen at intel.com>
Date:   Mon Jun 26 10:43:15 2017 +0300

i965: Separate gen < 8 and gen >= 8 paths explicitly in wrap_mode()

Makes coverity happier.

Fix indentation in gen >= 8 block while at it.

CID: 1413020
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Eric Engestrom <eric at engestrom.ch>
Signed-off-by: Topi Pohjolainen <topi.pohjolainen at intel.com>

---

 src/mesa/drivers/dri/i965/genX_state_upload.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
index 3f8a7265db..d65b468863 100644
--- a/src/mesa/drivers/dri/i965/genX_state_upload.c
+++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
@@ -4455,9 +4455,8 @@ translate_wrap_mode(struct brw_context *brw, GLenum wrap, bool using_nearest)
        *
        * Gen8+ supports this natively.
        */
-         return TCM_HALF_BORDER;
-#endif
-
+      return TCM_HALF_BORDER;
+#else
       /* On Gen4-7.5, we clamp the coordinates in the fragment shader
        * and set clamp_border here, which gets the result desired.
        * We just use clamp(_to_edge) for nearest, because for nearest
@@ -4468,6 +4467,7 @@ translate_wrap_mode(struct brw_context *brw, GLenum wrap, bool using_nearest)
          return TCM_CLAMP;
       else
          return TCM_CLAMP_BORDER;
+#endif
    case GL_CLAMP_TO_EDGE:
       return TCM_CLAMP;
    case GL_CLAMP_TO_BORDER:




More information about the mesa-commit mailing list