KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

i965/fs: Factor out source components calculation to a separate method. 

This cleans up fs_inst::regs_read() slightly by disentangling the
calculation of "components" from the handling of message payload
arguments.  This will also simplify the SIMD lowering and logical send
message lowering passes, because it will avoid expressions like
'regs_read * REG_SIZE / component_size' which are not only ugly, they
may be inaccurate because regs_read rounds up the result to the
closest register multiple so they could give incorrect results when
the component size is lower than one register (e.g. uniforms).  This
didn't seem to be a problem right now because all such expressions
happen to be dealing with per-channel GRFs only currently, but that's
by no means obvious so better be safe than sorry.

v2: Split PIXEL_X/Y and LINTERP into separate case blocks.

Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
This commit is contained in:
Francisco Jerez 2015-07-21 17:28:39 +03:00
parent 24d74b6688
commit fb7eba97d7
2 changed files with 24 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
src/mesa/drivers/dri/i965/brw_fs.cpp
View File

@ -663,10 +663,29 @@ fs_inst::is_partial_write() const
!this->dst.is_contiguous());
}
unsigned
fs_inst::components_read(unsigned i) const
{
switch (opcode) {
case FS_OPCODE_LINTERP:
if (i == 0)
return 2;
else
return 1;
case FS_OPCODE_PIXEL_X:
case FS_OPCODE_PIXEL_Y:
assert(i == 0);
return 2;
default:
return 1;
}
}
int
fs_inst::regs_read(int arg) const
{
unsigned components = 1;
switch (opcode) {
case FS_OPCODE_FB_WRITE:
case SHADER_OPCODE_URB_WRITE_SIMD8:
@ -688,15 +707,8 @@ fs_inst::regs_read(int arg) const
break;
case FS_OPCODE_LINTERP:
if (arg == 0)
return exec_size / 4;
else
if (arg == 1)
return 1;
case FS_OPCODE_PIXEL_X:
case FS_OPCODE_PIXEL_Y:
if (arg == 0)
components = 2;
break;
case SHADER_OPCODE_LOAD_PAYLOAD:
@ -720,7 +732,8 @@ fs_inst::regs_read(int arg) const
return 1;
case GRF:
case HW_REG:
return DIV_ROUND_UP(components * src[arg].component_size(exec_size),
return DIV_ROUND_UP(components_read(arg) *
src[arg].component_size(exec_size),
REG_SIZE);
case MRF:
unreachable("MRF registers are not allowed as sources");

1
src/mesa/drivers/dri/i965/brw_ir_fs.h
View File

@ -201,6 +201,7 @@ public:
bool is_send_from_grf() const;
bool is_partial_write() const;
bool is_copy_payload(const brw::simple_allocator &grf_alloc) const;
unsigned components_read(unsigned i) const;
int regs_read(int arg) const;
bool can_do_source_mods(const struct brw_device_info *devinfo);
bool has_side_effects() const;