Mesa (master): i965: Drop has_pln checks in unlit centroid workaround.

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


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

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

i965: Drop has_pln checks in unlit centroid workaround.

The unlit centroid workaround starts being necessary on Gen6, which
is the first platform with multisampling.  PLN exists on G45+, so all
platforms which need this workaround have PLN.

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 | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 446cf3a..940f425 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -1185,16 +1185,13 @@ fs_visitor::emit_general_interpolation(fs_reg *attr, const char *name,
                                delta_xy[centroid_to_pixel(bary)], interp);
                inst->predicate = BRW_PREDICATE_NORMAL;
                inst->predicate_inverse = true;
-               if (devinfo->has_pln)
-                  inst->no_dd_clear = true;
+               inst->no_dd_clear = true;
 
                inst = bld.emit(FS_OPCODE_LINTERP, *attr,
                                delta_xy[bary], interp);
                inst->predicate = BRW_PREDICATE_NORMAL;
                inst->predicate_inverse = false;
-               if (devinfo->has_pln)
-                  inst->no_dd_check = true;
-
+               inst->no_dd_check = true;
             } else {
                bld.emit(FS_OPCODE_LINTERP, *attr, delta_xy[bary], interp);
             }




More information about the mesa-commit mailing list