[llvm] [AMDGPU][NFC] Simplify t16/fake16 TableGen definitions. (PR #122693)
Joe Nash via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 13 07:55:45 PST 2025
================
@@ -973,12 +973,10 @@ multiclass VOP1_Real_FULL_with_name_gfx11_gfx12<bits<9> op, string opName,
VOP1_Real_FULL_with_name<GFX11Gen, op, opName, asmName>,
VOP1_Real_FULL_with_name<GFX12Gen, op, opName, asmName>;
-multiclass VOP1_Real_FULL_t16_and_fake16_gfx11_gfx12<bits<9> op, string asmName,
- string opName = NAME> {
- defm opName#"_t16" :
- VOP1_Real_FULL_with_name_gfx11_gfx12<op, opName#"_t16", asmName>;
- defm opName#"_fake16":
- VOP1_Real_FULL_with_name_gfx11_gfx12<op, opName#"_fake16", asmName>;
+multiclass VOP1_Real_FULL_t16_and_fake16_gfx11_gfx12<bits<9> op> {
----------------
Sisyph wrote:
Though this multiclass does not support any renamed instructions, I think it would be good to support a uniform interface if it is renamed or not (in other VOP2, VOP3 etc). So instead of deleting asmName parameter, give it a default.
https://github.com/llvm/llvm-project/pull/122693
More information about the llvm-commits
mailing list