diff options
author | Francisco Jerez <[email protected]> | 2016-05-20 00:38:17 -0700 |
---|---|---|
committer | Francisco Jerez <[email protected]> | 2016-05-29 23:41:38 -0700 |
commit | a5b4f63c1593cdcbc253cce2838c85b2fd796dac (patch) | |
tree | 70d5199931b1ccf62203003bf2b83ec381e7ba60 /src/mesa/state_tracker/st_cache.h | |
parent | a0d9aed2682f78626f467cbc2b7fc3185d9f9034 (diff) |
i965/fs: Implement opt_sampler_eot() in terms of logical sends.
This makes the whole LOAD_PAYLOAD munging unnecessary which simplifies
the code and will allow the optimization to succeed in more cases
independent of whether the LOAD_PAYLOAD instruction can be found or
not.
The following patch is squashed in:
SQUASH: i965/fs: Add basic dataflow check to opt_sampler_eot().
The sampler EOT optimization pass naively assumes that the texturing
instruction provides all the data used by the FB write just because
they're standing next to each other. The least we should be checking
is whether the source and destination regions of the FB write and
texturing instructions match. Without this the previous seemingly
harmless patch would have caused opt_sampler_eot() to misoptimize a
shader from dota-2 causing DCE to eliminate all of its 78 instructions
except for the final sampler EOT message (!).
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src/mesa/state_tracker/st_cache.h')
0 files changed, 0 insertions, 0 deletions