diff options
| author | Ian Romanick <[email protected]> | 2018-06-05 16:02:25 -0700 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2018-06-14 11:28:12 -0700 |
| commit | 37bd9ccd21b860d2b5ffea7e1f472ec83b68b43b (patch) | |
| tree | e4fd24ef618e4a141b764b206aef9deabad63eb8 /src/intel/compiler | |
| parent | 461a5c899c08064467abb635536381a5a5659280 (diff) | |
glsl: Don't copy propagate elements from SSBO or shared variables either
Since SSBOs can be written by a different GPU thread, copy propagating a
read can cause the value to magically change. SSBO reads are also very
expensive, so doing it twice will be slower.
The same shader was helped by this patch and the previous.
Haswell, Broadwell, and Skylake had similar results. (Skylake shown)
total instructions in shared programs: 14399119 -> 14399113 (<.01%)
instructions in affected programs: 683 -> 677 (-0.88%)
helped: 1
HURT: 0
total cycles in shared programs: 532973113 -> 532971865 (<.01%)
cycles in affected programs: 524666 -> 523418 (-0.24%)
helped: 1
HURT: 0
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Caio Marcelo de Oliveira Filho <[email protected]>
Cc: [email protected]
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106774
Diffstat (limited to 'src/intel/compiler')
0 files changed, 0 insertions, 0 deletions