Skip to content

Commit b2379bd

Browse files
authored
[AMDGPU] Support bottom-up postRA scheduing. (#135295)
Solely relying on top‑down scheduling can underutilize hardware, since long‑latency instructions often end up scheduled too late and their latency isn’t well hidden. Adding bottom‑up post‑RA scheduling lets us move those instructions earlier, which improves latency hiding and yields roughly a 2% performance gain on key benchmarks.
1 parent 2d7f53b commit b2379bd

File tree

2 files changed

+163
-1
lines changed

2 files changed

+163
-1
lines changed

llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -417,7 +417,8 @@ void GCNHazardRecognizer::AdvanceCycle() {
417417
}
418418

419419
void GCNHazardRecognizer::RecedeCycle() {
420-
llvm_unreachable("hazard recognizer does not support bottom-up scheduling.");
420+
assert(!IsHazardRecognizerMode &&
421+
"Bottom-up scheduling shouldn't run in hazard recognizer mode");
421422
}
422423

423424
//===----------------------------------------------------------------------===//

0 commit comments

Comments
 (0)