aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/CodeGen/DFAPacketizer.cpp
diff options
context:
space:
mode:
authorJay Foad <jay.foad@gmail.com>2019-09-06 10:07:28 +0000
committerJay Foad <jay.foad@gmail.com>2019-09-06 10:07:28 +0000
commit6c0204c794ba143ce6817f695e03fd9f49cd4ed8 (patch)
tree1ce58102bd39e3c4b6e0962fa1f4321975755608 /llvm/lib/CodeGen/DFAPacketizer.cpp
parent68df9dc09818289ac5b40d0916bb479a358a8e18 (diff)
downloadllvm-6c0204c794ba143ce6817f695e03fd9f49cd4ed8.zip
llvm-6c0204c794ba143ce6817f695e03fd9f49cd4ed8.tar.gz
llvm-6c0204c794ba143ce6817f695e03fd9f49cd4ed8.tar.bz2
[AMDGPU] Mark s_barrier as having side effects but not accessing memory.
Summary: This fixes poor scheduling in a function containing a barrier and a few load instructions. Without this fix, ScheduleDAGInstrs::buildSchedGraph adds an artificial edge in the dependency graph from the barrier instruction to the exit node representing live-out latency, with a latency of about 500 cycles. Because of this it thinks the critical path through the graph also has a latency of about 500 cycles. And because of that it does not think that any of the load instructions are on the critical path, so it schedules them with no regard for their (80 cycle) latency, which gives poor results. Reviewers: arsenm, dstuttard, tpr, nhaehnle Subscribers: kzhuravl, jvesely, wdng, yaxunl, t-tye, hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D67218 llvm-svn: 371192
Diffstat (limited to 'llvm/lib/CodeGen/DFAPacketizer.cpp')
0 files changed, 0 insertions, 0 deletions