From 12011b5217929ef8a56c2099c6f3233934ea4fbc Mon Sep 17 00:00:00 2001 From: Jay Foad Date: Tue, 27 Apr 2021 17:03:22 +0100 Subject: [PATCH] [AMDGPU] GCNHazardRecognizer: ignore all meta instructions This is hopefully NFC, but should be more robust in ignoring all instructions that should be ignored, instead of just some of them. Differential Revision: https://reviews.llvm.org/D101372 --- llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp b/llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp index f5db222fc05b..82e1c58210b2 100644 --- a/llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp +++ b/llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp @@ -348,8 +348,7 @@ void GCNHazardRecognizer::AdvanceCycle() { // Do not track non-instructions which do not affect the wait states. // If included, these instructions can lead to buffer overflow such that // detectable hazards are missed. - if (CurrCycleInstr->isImplicitDef() || CurrCycleInstr->isDebugInstr() || - CurrCycleInstr->isKill()) { + if (CurrCycleInstr->isMetaInstruction()) { CurrCycleInstr = nullptr; return; }