summaryrefslogtreecommitdiffstats
path: root/src/intel/isl
diff options
context:
space:
mode:
authorKenneth Graunke <[email protected]>2017-10-17 23:19:20 -0700
committerKenneth Graunke <[email protected]>2017-10-19 10:19:20 -0700
commit68f69ebdccca16246bafd86e86ed80fab369009b (patch)
treeed4fd481afe0f61e2cff21b7f979d0c03e94f6ac /src/intel/isl
parent3d112a7cd49656da3dde5f9b476b6ddc2d984528 (diff)
i965: Use is_scheduling_barrier instead of schedule_node::is_barrier.
Commit a73116ecc60414ade89802150b tried to make add_barrier_deps() walk to the next barrier, and stop. To accomplish that, it added an is_barrier flag. Unfortunately, this only works half of the time. The issue is that add_barrier_deps() walks both backward (to the previous barrier), and forward (to the next barrier). It also sets is_barrier. Assuming that we're processing instructions in forward order, this means that is_barrier will be set for previous instructions, but not future ones. So we'll never see it, and walk further than we need to. dEQP-GLES31.functional.ssbo.layout.random.all_shared_buffer.23 now compiles its shaders in 3.6 seconds instead of 3.3 minutes. Reviewed-by: Matt Turner <[email protected]> Tested-by: Pallavi G <[email protected]>
Diffstat (limited to 'src/intel/isl')
0 files changed, 0 insertions, 0 deletions