summaryrefslogtreecommitdiff
path: root/src/amd
diff options
context:
space:
mode:
authorIan Romanick <ian.d.romanick@intel.com>2018-06-05 16:02:25 -0700
committerDylan Baker <dylan@pnwbakers.com>2018-06-15 13:55:04 -0700
commita4609fe84f6c773923e90b43ad7a2e17dfe56a7a (patch)
tree773fffcc00144f77f1526d46f445b55281e657a5 /src/amd
parentf3ec346ab19d25de43a3fd14f533fbc40522d8ea (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 <ian.d.romanick@intel.com> Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com> Cc: mesa-stable@lists.freedesktop.org Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106774 (cherry picked from commit 37bd9ccd21b860d2b5ffea7e1f472ec83b68b43b)
Diffstat (limited to 'src/amd')
0 files changed, 0 insertions, 0 deletions