[llvm] [AMDGPU] Simplify renamed BUF instruction definitions. NFC. (PR #84503)

Joe Nash via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 8 08:08:19 PST 2024


================
@@ -2895,28 +2887,25 @@ multiclass MTBUF_Real_AllAddr_gfx11_gfx12_Renamed_Impl<bits<4> op, string real_n
   defm _VBUFFER_OFFSET : VBUFFER_MTBUF_Real_gfx12<op, real_name>;
 }
 
-multiclass MTBUF_Real_AllAddr_gfx11_gfx12<bits<4> op>
- : MTBUF_Real_AllAddr_gfx11_gfx12_Renamed_Impl<op, get_BUF_ps<NAME>.Mnemonic>;
-
-
-multiclass MTBUF_Real_AllAddr_gfx11_gfx12_Renamed<bits<4> op, string real_name>
-  : MTBUF_Real_AllAddr_gfx11_gfx12_Renamed_Impl<op, real_name> {
+multiclass MTBUF_Real_AllAddr_gfx11_gfx12<bits<4> op, string real_name = !tolower(NAME)>
----------------
Sisyph wrote:

```suggestion
multiclass MTBUF_Real_AllAddr_gfx11_gfx12<bits<4> op, string real_name = get_BUF_ps<NAME>.Mnemonic
```

It is assumed now that !tolower(NAME) == !tolower(NAME), but I think there is no need to make that assumption. Using the suggested formulation avoids it.

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


More information about the llvm-commits mailing list