[Mesa-dev] [PATCH 2/4] i965: Drop has_pln checks in unlit centroid workaround.

Kenneth Graunke kenneth at whitecape.org
Sat Jul 16 06:43:13 UTC 2016


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>
---
 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);
             }
-- 
2.9.0



More information about the mesa-dev mailing list