[llvm] [AMDGPU] Fix comment on DelayInfo::advance (PR #146718)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 2 08:09:03 PDT 2025


https://github.com/jayfoad created https://github.com/llvm/llvm-project/pull/146718

None

>From 4c016fc0fcf502e7c4aedf800eeaa3f1ae2ffcc2 Mon Sep 17 00:00:00 2001
From: Jay Foad <jay.foad at amd.com>
Date: Wed, 2 Jul 2025 16:04:23 +0100
Subject: [PATCH] [AMDGPU] Fix comment on DelayInfo::advance

---
 llvm/lib/Target/AMDGPU/AMDGPUInsertDelayAlu.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/llvm/lib/Target/AMDGPU/AMDGPUInsertDelayAlu.cpp b/llvm/lib/Target/AMDGPU/AMDGPUInsertDelayAlu.cpp
index 383b5c0e9dcd5..44eaebffb70dc 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUInsertDelayAlu.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUInsertDelayAlu.cpp
@@ -156,10 +156,9 @@ class AMDGPUInsertDelayAlu {
       SALUCycles = std::max(SALUCycles, RHS.SALUCycles);
     }
 
-    // Update this DelayInfo after issuing an instruction. IsVALU should be 1
-    // when issuing a (non-TRANS) VALU, else 0. IsTRANS should be 1 when issuing
-    // a TRANS, else 0. Cycles is the number of cycles it takes to issue the
-    // instruction.  Return true if there is no longer any useful delay info.
+    // Update this DelayInfo after issuing an instruction of the specified type.
+    // Cycles is the number of cycles it takes to issue the instruction.  Return
+    // true if there is no longer any useful delay info.
     bool advance(DelayType Type, unsigned Cycles) {
       bool Erase = true;
 



More information about the llvm-commits mailing list