From: Francisco Jerez <curroje...@riseup.net> This generalizes the unlit centroid workaround so it's less code and now supports SIMD32.
Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net> --- src/intel/compiler/brw_fs_visitor.cpp | 22 ++++++++-------------- 1 file changed, 8 insertions(+), 14 deletions(-) diff --git a/src/intel/compiler/brw_fs_visitor.cpp b/src/intel/compiler/brw_fs_visitor.cpp index a0ebd60..ed1348b 100644 --- a/src/intel/compiler/brw_fs_visitor.cpp +++ b/src/intel/compiler/brw_fs_visitor.cpp @@ -281,21 +281,15 @@ fs_visitor::emit_interpolation_setup_gen6() */ bld.emit(FS_OPCODE_MOV_DISPATCH_TO_FLAGS); - set_predicate_inv(BRW_PREDICATE_NORMAL, true, - bld.half(0).MOV(horiz_offset(delta_xy[i], 0), - horiz_offset(pixel_delta_xy, 0))); - set_predicate_inv(BRW_PREDICATE_NORMAL, true, - bld.half(0).MOV(horiz_offset(delta_xy[i], 8), - horiz_offset(pixel_delta_xy, 8))); - if (dispatch_width == 16) { - set_predicate_inv(BRW_PREDICATE_NORMAL, true, - bld.half(1).MOV(horiz_offset(delta_xy[i], 16), - horiz_offset(pixel_delta_xy, 16))); - set_predicate_inv(BRW_PREDICATE_NORMAL, true, - bld.half(1).MOV(horiz_offset(delta_xy[i], 24), - horiz_offset(pixel_delta_xy, 24))); + for (unsigned q = 0; q < dispatch_width / 8; q++) { + for (unsigned c = 0; c < 2; c++) { + const unsigned idx = c + (q & 2) + (q & 1) * dispatch_width / 8; + set_predicate_inv( + BRW_PREDICATE_NORMAL, true, + bld.half(q).MOV(horiz_offset(delta_xy[i], idx * 8), + horiz_offset(pixel_delta_xy, idx * 8))); + } } - assert(dispatch_width != 32); /* not implemented yet */ } } } -- 2.5.0.400.gff86faf _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev