If Matt concerns about the validation rule are solved this is. Reviewed-by: Jose Maria Casanova Crespo <jmcasan...@igalia.com>
El 15/1/19 a las 17:58, Jason Ekstrand escribió: > Previously, we only applied the fix to shaders with a dispatch mode of > SIMD8 but the code it relies on for SIMD16 mode only applies to SIMD16 > instructions. If you have a SIMD8 instruction in a SIMD16 shader, > neither would trigger and the restriction could still be hit. > > Cc: Jose Maria Casanova Crespo <jmcasan...@igalia.com> > Fixes: 232ed8980217dd "i965/fs: Register allocator shoudn't use grf127..." > --- > src/intel/compiler/brw_fs_reg_allocate.cpp | 13 ++++++------- > 1 file changed, 6 insertions(+), 7 deletions(-) > > diff --git a/src/intel/compiler/brw_fs_reg_allocate.cpp > b/src/intel/compiler/brw_fs_reg_allocate.cpp > index 5db5242452e..ec743f9b5bf 100644 > --- a/src/intel/compiler/brw_fs_reg_allocate.cpp > +++ b/src/intel/compiler/brw_fs_reg_allocate.cpp > @@ -667,15 +667,14 @@ fs_visitor::assign_regs(bool allow_spilling, bool > spill_all) > * messages adding a node interference to the grf127_send_hack_node. > * This node has a fixed asignment to grf127. > * > - * We don't apply it to SIMD16 because previous code avoids any > register > - * overlap between sources and destination. > + * We don't apply it to SIMD16 instructions because previous code > avoids > + * any register overlap between sources and destination. > */ > ra_set_node_reg(g, grf127_send_hack_node, 127); > - if (dispatch_width == 8) { > - foreach_block_and_inst(block, fs_inst, inst, cfg) { > - if (inst->is_send_from_grf() && inst->dst.file == VGRF) > - ra_add_node_interference(g, inst->dst.nr, > grf127_send_hack_node); > - } > + foreach_block_and_inst(block, fs_inst, inst, cfg) { > + if (inst->exec_size < 16 && inst->is_send_from_grf() && > + inst->dst.file == VGRF) > + ra_add_node_interference(g, inst->dst.nr, grf127_send_hack_node); > } > > if (spilled_any_registers) { > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev