diff options
| author | Matt Turner <mattst88@gmail.com> | 2015-10-26 17:52:57 -0700 |
|---|---|---|
| committer | Matt Turner <mattst88@gmail.com> | 2015-11-13 11:27:51 -0800 |
| commit | b3315a6f56fb93f2884168cbf9358b2606641db5 (patch) | |
| tree | 293de5d4f7a27381b8fc220f35be1862749a18a3 /src/mesa/drivers/dri/i965/brw_fs_cse.cpp | |
| parent | 4b0fbebf024e564c195f3ce94e1ce43a3d6442ea (diff) | |
| download | external_mesa3d-b3315a6f56fb93f2884168cbf9358b2606641db5.zip external_mesa3d-b3315a6f56fb93f2884168cbf9358b2606641db5.tar.gz external_mesa3d-b3315a6f56fb93f2884168cbf9358b2606641db5.tar.bz2 | |
i965: Replace HW_REG with ARF/FIXED_GRF.
HW_REGs are (were!) kind of awful. If the file was HW_REG, you had to
look at different fields for type, abs, negate, writemask, swizzle, and
a second file. They also caused annoying problems like immediate sources
being considered scheduling barriers (commit 6148e94e2) and other such
nonsense.
Instead use ARF/FIXED_GRF/MRF for fixed registers in those files.
After a sufficient amount of time has passed since "GRF" was used, we
can rename FIXED_GRF -> GRF, but doing so now would make rebasing awful.
Reviewed-by: Emil Velikov <emil.velikov@collabora.co.uk>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_fs_cse.cpp')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp index ee31cb6..8c67caf 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp @@ -226,7 +226,8 @@ fs_visitor::opt_cse_local(bblock_t *block) foreach_inst_in_block(fs_inst, inst, block) { /* Skip some cases. */ if (is_expression(this, inst) && !inst->is_partial_write() && - (inst->dst.file != HW_REG || inst->dst.is_null())) + ((inst->dst.file != ARF && inst->dst.file != FIXED_GRF) || + inst->dst.is_null())) { bool found = false; bool negate = false; |