diff options
| author | Francisco Jerez <currojerez@riseup.net> | 2016-09-01 19:34:18 -0700 |
|---|---|---|
| committer | Francisco Jerez <currojerez@riseup.net> | 2016-09-14 14:50:55 -0700 |
| commit | b42c13a5b8ac7d643bbf4c1592607811a81b4ebb (patch) | |
| tree | f8de2847535fa7899339b11040c6885ec1124d56 /src/mesa/drivers/dri/i965/brw_fs.cpp | |
| parent | 32d67923b28e55cfe23883510509644e788003f0 (diff) | |
i965/fs: Drop fs_inst::overwrites_reg() in favor of regions_overlap().
fs_inst::overwrites_reg is rather easy to misuse because it cannot
tell how large the register region starting at 'reg' is, so in cases
where the destination region starts after 'reg' it may give a
misleading result. regions_overlap() is somewhat more verbose to use
but handles arbitrary overlap correctly so it should generally be used
instead.
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_fs.cpp')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index f5ae60372f6..a52e5a37950 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp @@ -241,12 +241,6 @@ fs_inst::equals(fs_inst *inst) const } bool -fs_inst::overwrites_reg(const fs_reg ®) const -{ - return reg.in_range(dst, DIV_ROUND_UP(size_written, REG_SIZE)); -} - -bool fs_inst::is_send_from_grf() const { switch (opcode) { |