diff options
| author | Eric Anholt <[email protected]> | 2013-03-19 17:45:02 -0700 |
|---|---|---|
| committer | Eric Anholt <[email protected]> | 2013-04-01 16:17:26 -0700 |
| commit | 62501c3af85089b423218a41a2e2433ac849c2d3 (patch) | |
| tree | 09b6ed2ac149fb41d91a204b7742b122ab466e86 | |
| parent | 70b27e0e4b5d15e575ea477d63c0f6cb19d645c2 (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 <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=61554
NOTE: This is a candidate for the 9.1 branch.
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 2 |
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 79169669e6a..2a8fd0be2e7 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) { |