Skip to content

Commit c3c9b84

Browse files
committed
AMDGPU: Reorder arguments of DS_Real_gfx12
This helps shrink the diff in a future change.
1 parent 2bb4e1e commit c3c9b84

File tree

1 file changed

+26
-15
lines changed

1 file changed

+26
-15
lines changed

llvm/lib/Target/AMDGPU/DSInstructions.td

Lines changed: 26 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1360,8 +1360,10 @@ class Base_DS_Real_gfx6_gfx7_gfx10_gfx11_gfx12<bits<8> op, DS_Pseudo ps, int ef,
13601360
// GFX12.
13611361
//===----------------------------------------------------------------------===//
13621362

1363-
multiclass DS_Real_gfx12<bits<8> op, string name = !tolower(NAME)> {
1364-
defvar ps = !cast<DS_Pseudo>(NAME);
1363+
multiclass DS_Real_gfx12<bits<8> op,
1364+
DS_Pseudo ps = !cast<DS_Pseudo>(NAME),
1365+
string name = !tolower(NAME)> {
1366+
13651367
let AssemblerPredicate = isGFX12Plus in {
13661368
let DecoderNamespace = "GFX12" in
13671369
def _gfx12 :
@@ -1372,14 +1374,20 @@ multiclass DS_Real_gfx12<bits<8> op, string name = !tolower(NAME)> {
13721374
} // End AssemblerPredicate
13731375
}
13741376

1375-
defm DS_MIN_F32 : DS_Real_gfx12<0x012, "ds_min_num_f32">;
1376-
defm DS_MAX_F32 : DS_Real_gfx12<0x013, "ds_max_num_f32">;
1377-
defm DS_MIN_RTN_F32 : DS_Real_gfx12<0x032, "ds_min_num_rtn_f32">;
1378-
defm DS_MAX_RTN_F32 : DS_Real_gfx12<0x033, "ds_max_num_rtn_f32">;
1379-
defm DS_MIN_F64 : DS_Real_gfx12<0x052, "ds_min_num_f64">;
1380-
defm DS_MAX_F64 : DS_Real_gfx12<0x053, "ds_max_num_f64">;
1381-
defm DS_MIN_RTN_F64 : DS_Real_gfx12<0x072, "ds_min_num_rtn_f64">;
1382-
defm DS_MAX_RTN_F64 : DS_Real_gfx12<0x073, "ds_max_num_rtn_f64">;
1377+
// Helper to avoid repeating the pseudo-name if we only need to set
1378+
// the gfx12 name.
1379+
multiclass DS_Real_gfx12_with_name<bits<8> op, string name> {
1380+
defm "" : DS_Real_gfx12<op, !cast<DS_Pseudo>(NAME), name>;
1381+
}
1382+
1383+
defm DS_MIN_F32 : DS_Real_gfx12_with_name<0x012, "ds_min_num_f32">;
1384+
defm DS_MAX_F32 : DS_Real_gfx12_with_name<0x013, "ds_max_num_f32">;
1385+
defm DS_MIN_RTN_F32 : DS_Real_gfx12_with_name<0x032, "ds_min_num_rtn_f32">;
1386+
defm DS_MAX_RTN_F32 : DS_Real_gfx12_with_name<0x033, "ds_max_num_rtn_f32">;
1387+
defm DS_MIN_F64 : DS_Real_gfx12_with_name<0x052, "ds_min_num_f64">;
1388+
defm DS_MAX_F64 : DS_Real_gfx12_with_name<0x053, "ds_max_num_f64">;
1389+
defm DS_MIN_RTN_F64 : DS_Real_gfx12_with_name<0x072, "ds_min_num_rtn_f64">;
1390+
defm DS_MAX_RTN_F64 : DS_Real_gfx12_with_name<0x073, "ds_max_num_rtn_f64">;
13831391
defm DS_COND_SUB_U32 : DS_Real_gfx12<0x098>;
13841392
defm DS_SUB_CLAMP_U32 : DS_Real_gfx12<0x099>;
13851393
defm DS_COND_SUB_RTN_U32 : DS_Real_gfx12<0x0a8>;
@@ -1395,7 +1403,7 @@ defm DS_LOAD_TR6_B96 : DS_Real_gfx12<0x0fb>;
13951403
defm DS_LOAD_TR16_B128 : DS_Real_gfx12<0x0fc>;
13961404
defm DS_LOAD_TR8_B64 : DS_Real_gfx12<0x0fd>;
13971405

1398-
defm DS_BVH_STACK_RTN_B32 : DS_Real_gfx12<0x0e0,
1406+
defm DS_BVH_STACK_RTN_B32 : DS_Real_gfx12_with_name<0x0e0,
13991407
"ds_bvh_stack_push4_pop1_rtn_b32">;
14001408
defm DS_BVH_STACK_PUSH8_POP1_RTN_B32 : DS_Real_gfx12<0x0e1>;
14011409
defm DS_BVH_STACK_PUSH8_POP2_RTN_B64 : DS_Real_gfx12<0x0e2>;
@@ -1424,8 +1432,8 @@ def : MnemonicAlias<"ds_load_tr_b128", "ds_load_tr16_b128">, Requires<[isGFX1250
14241432
// GFX11.
14251433
//===----------------------------------------------------------------------===//
14261434

1427-
multiclass DS_Real_gfx11<bits<8> op, string name = !tolower(NAME)> {
1428-
defvar ps = !cast<DS_Pseudo>(NAME);
1435+
multiclass DS_Real_gfx11<bits<8> op, DS_Pseudo ps = !cast<DS_Pseudo>(NAME),
1436+
string name = !tolower(NAME)> {
14291437
let AssemblerPredicate = isGFX11Only in {
14301438
let DecoderNamespace = "GFX11" in
14311439
def _gfx11 :
@@ -1436,8 +1444,11 @@ multiclass DS_Real_gfx11<bits<8> op, string name = !tolower(NAME)> {
14361444
} // End AssemblerPredicate
14371445
}
14381446

1439-
multiclass DS_Real_gfx11_gfx12<bits<8> op, string name = !tolower(NAME)>
1440-
: DS_Real_gfx11<op, name>, DS_Real_gfx12<op, name>;
1447+
multiclass DS_Real_gfx11_gfx12<bits<8> op,
1448+
string name = !tolower(NAME),
1449+
DS_Pseudo ps = !cast<DS_Pseudo>(NAME)>
1450+
: DS_Real_gfx11<op, ps, name>,
1451+
DS_Real_gfx12<op, ps, name>;
14411452

14421453
defm DS_WRITE_B32 : DS_Real_gfx11_gfx12<0x00d, "ds_store_b32">;
14431454
defm DS_WRITE2_B32 : DS_Real_gfx11_gfx12<0x00e, "ds_store_2addr_b32">;

0 commit comments

Comments
 (0)