diff options
| author | Jason Ekstrand <jason.ekstrand@intel.com> | 2019-01-15 10:53:44 -0600 |
|---|---|---|
| committer | Dylan Baker <dylan@pnwbakers.com> | 2019-02-04 09:16:21 -0800 |
| commit | c6649ca94d07daa605814b243706e9ba4ca29576 (patch) | |
| tree | 700ea8e8cc23e2354d3f68ad0f0f411c519a710d /src/intel | |
| parent | 89f84f98e0451629e44dcb0a7cdba10b60515bf6 (diff) | |
| download | mesa-c6649ca94d07daa605814b243706e9ba4ca29576.tar.gz mesa-c6649ca94d07daa605814b243706e9ba4ca29576.tar.bz2 mesa-c6649ca94d07daa605814b243706e9ba4ca29576.zip | |
intel/fs: Do the grf127 hack on SIMD8 instructions in SIMD16 mode
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.
Fixes: 232ed8980217dd "i965/fs: Register allocator shoudn't use grf127..."
Reviewed-by: Jose Maria Casanova Crespo <jmcasanova@igalia.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
(cherry picked from commit b4f0d062cd12b4f675bac900ac41d1085a79239a)
Diffstat (limited to 'src/intel')
| -rw-r--r-- | src/intel/compiler/brw_fs_reg_allocate.cpp | 13 |
1 files 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 6961cb1caf4..b3825f1ef8c 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) { |