Mesa (master): i965: Drop VARYING_SLOT_FACE special case in barycentric setup.

Kenneth Graunke kwg at kemper.freedesktop.org
Mon Jul 18 03:02:01 UTC 2016


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Thu Jul 14 11:48:01 2016 -0700

i965: Drop VARYING_SLOT_FACE special case in barycentric setup.

glsl_to_nir always produces a system value for gl_FrontFacing, rather
than an input.  So there should never be an input with this slot,
making this code dead.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Timothy Arceri <timothy.arceri at collabora.com>

---

 src/mesa/drivers/dri/i965/brw_fs.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
index c907725..446cf3a 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -6341,9 +6341,8 @@ brw_compute_barycentric_interp_modes(const struct brw_device_info *devinfo,
    unsigned barycentric_interp_modes = 0;
 
    nir_foreach_variable(var, &shader->inputs) {
-      /* Ignore WPOS and FACE, because they don't require interpolation. */
-      if (var->data.location == VARYING_SLOT_POS ||
-          var->data.location == VARYING_SLOT_FACE)
+      /* Ignore WPOS; it doesn't require interpolation. */
+      if (var->data.location == VARYING_SLOT_POS)
          continue;
 
       /* Flat inputs don't need barycentric modes. */




More information about the mesa-commit mailing list