[llvm] [AMDGPU] Add GFX12 wave register names with WAVE_ prefix (PR #144352)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 23 07:46:26 PDT 2025


================
@@ -27,21 +25,22 @@ template <size_t N>
 static StringRef getNameFromOperandTable(const CustomOperand (&Table)[N],
                                          unsigned Encoding,
                                          const MCSubtargetInfo &STI) {
-  auto isValidIndexForEncoding = [&](size_t Idx) {
-    return Idx < N && Table[Idx].Encoding == Encoding &&
-           !Table[Idx].Name.empty() &&
-           (!Table[Idx].Cond || Table[Idx].Cond(STI));
+  auto IsValid = [&](const CustomOperand &Entry) {
+    return Entry.Encoding == Encoding && !Entry.Name.empty() &&
----------------
jayfoad wrote:

Name is never empty, is it?
```suggestion
    return Entry.Encoding == Encoding &&
```

https://github.com/llvm/llvm-project/pull/144352


More information about the llvm-commits mailing list