fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-03 05:38:16 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

i965/fs: Take into account misalignment in regs_written() and regs_read().

Browse source 
There was a workaround for this in fs_inst::size_read() for the
SHADER_OPCODE_MOV_INDIRECT instruction and FIXED_GRF register file
*only*.  We should take this possibility into account for the sources
and destinations of all instructions on all optimization passes that
need to quantize dataflow in 32B increments by adding the amount of
misalignment to the size read or written from the regs_read() and
regs_written() helpers respectively.

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
This commit is contained in:
Francisco Jerez 2016-09-07 14:36:32 -07:00
parent e540045df5
commit 717d8efd58
2 changed files with 5 additions and 29 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
src/mesa/drivers/dri/i965/brw_fs.cpp
View file

@ -863,31 +863,7 @@ fs_inst::size_read(int arg) const
case SHADER_OPCODE_MOV_INDIRECT:
if (arg == 0) {
assert(src[2].file == IMM);
unsigned region_length = src[2].ud;
if (src[0].file == UNIFORM) {
assert(region_length % 4 == 0);
return region_length;
} else if (src[0].file == FIXED_GRF) {
/* If the start of the region is not register aligned, then
* there's some portion of the register that's technically
* unread at the beginning.
*
* However, the register allocator works in terms of whole
* registers, and does not use subnr. It assumes that the
* read starts at the beginning of the register, and extends
* regs_read() whole registers beyond that.
*
* To compensate, we extend the region length to include this
* unread portion at the beginning.
*/
if (src[0].subnr)
region_length += src[0].subnr;
return region_length;
} else {
assert(!"Invalid register file");
}
return src[2].ud;
}
break;

8
src/mesa/drivers/dri/i965/brw_ir_fs.h
View file

@ -435,9 +435,9 @@ set_saturate(bool saturate, fs_inst *inst)
inline unsigned
regs_written(const fs_inst *inst)
{
/* XXX - Take into account register-misaligned offsets correctly. */
assert(inst->dst.file != UNIFORM && inst->dst.file != IMM);
return DIV_ROUND_UP(inst->size_written -
return DIV_ROUND_UP(reg_offset(inst->dst) % REG_SIZE +
inst->size_written -
MIN2(inst->size_written, reg_padding(inst->dst)),
REG_SIZE);
}
@ -451,10 +451,10 @@ regs_written(const fs_inst *inst)
inline unsigned
regs_read(const fs_inst *inst, unsigned i)
{
/* XXX - Take into account register-misaligned offsets correctly. */
const unsigned reg_size =
inst->src[i].file == UNIFORM || inst->src[i].file == IMM ? 4 : REG_SIZE;
return DIV_ROUND_UP(inst->size_read(i) -
return DIV_ROUND_UP(reg_offset(inst->src[i]) % reg_size +
inst->size_read(i) -
MIN2(inst->size_read(i), reg_padding(inst->src[i])),
reg_size);
}