summaryrefslogtreecommitdiff
path: root/src/mesa/drivers/dri/i965/brw_fs_cse.cpp
diff options
context:
space:
mode:
authorEric Anholt <eric@anholt.net>2013-03-19 17:45:02 -0700
committerEric Anholt <eric@anholt.net>2013-04-01 16:17:26 -0700
commit62501c3af85089b423218a41a2e2433ac849c2d3 (patch)
tree09b6ed2ac149fb41d91a204b7742b122ab466e86 /src/mesa/drivers/dri/i965/brw_fs_cse.cpp
parent70b27e0e4b5d15e575ea477d63c0f6cb19d645c2 (diff)
i965/fs: Allow CSE on pre-gen7 varying-index uniform loads
All the other expression types allowed here have inst->mlen == 0, and this one has implied MRF writes for all of its payload, so nothing else in the implementation should need to change. Reduces SEND messages for loading from pull constants in kwin's Lanczos shader from 16 to 6. (Due to a deficiency in constant propagation, I can't use the hack I did in the previous commit to test the performance change) Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=61554 NOTE: This is a candidate for the 9.1 branch.
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_fs_cse.cpp')
-rw-r--r--src/mesa/drivers/dri/i965/brw_fs_cse.cpp2
1 files changed, 1 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 79169669e6..2a8fd0be2e 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp
@@ -97,7 +97,7 @@ fs_visitor::opt_cse_local(bblock_t *block, exec_list *aeb)
inst = (fs_inst *) inst->next) {
/* Skip some cases. */
- if (is_expression(inst) && !inst->predicate && inst->mlen == 0 &&
+ if (is_expression(inst) && !inst->predicate &&
!inst->force_uncompressed && !inst->force_sechalf &&
!inst->conditional_mod)
{