[AMDGPU] Don't implement GCNHazardRecognizer::PreEmitNoops(SUnit *)
authorJay Foad <jay.foad@amd.com>
Wed, 6 May 2020 10:16:57 +0000 (11:16 +0100)
committerJay Foad <jay.foad@amd.com>
Wed, 6 May 2020 15:11:19 +0000 (16:11 +0100)
commit29067aac46d4f53f75c2d7e07e1afb7cb36212be
tree3057de620d7b41894e157f596df7175958148693
parenta3e6e624c71d60509c84e13ea3ee96c0acde65c3
[AMDGPU] Don't implement GCNHazardRecognizer::PreEmitNoops(SUnit *)

When called from the post-RA scheduler, hazards have already been
handled by getHazardType returning NoopHazard, so PreEmitNoops always
returns zero. Remove it. NFC.

Historical note: PreEmitNoops was added to the hazard recognizer
interface as an optional feature to support dispatch group formation on
the POWER target:
http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20131202/197470.html
So it seems right that we shouldn't need to implement it.

We do still implement the other overload PreEmitNoops(MachineInstr *)
because that is used by the PostRAHazardRecognizer pass.

Differential Revision: https://reviews.llvm.org/D79476
llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp
llvm/lib/Target/AMDGPU/GCNHazardRecognizer.h