Skip to content

Commit 2326b2b

Browse files
authored
[AMDGPU] Remove unnecessary conditionality on atomic CSUB pseudo-ops (#69914)
The pseudo-ops for BUFFER_ATOMIC_CSUB and GLOBAL_ATOMIC_CSUB should not be conditional on FeatureAtomicCSubNoRtnInsts, as that feature rightly should only control whether or not the non-returning forms are available for instruction selection, not whether they exist or not. Change-Id: Icb2fcdcf6c7bc55e48b522d0eec8dd35637c768e
1 parent c836b4a commit 2326b2b

File tree

2 files changed

+2
-11
lines changed

2 files changed

+2
-11
lines changed

llvm/lib/Target/AMDGPU/BUFInstructions.td

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1047,14 +1047,9 @@ defm BUFFER_ATOMIC_DEC_X2 : MUBUF_Pseudo_Atomics <
10471047
>;
10481048

10491049
let SubtargetPredicate = HasGFX10_BEncoding in {
1050-
defm BUFFER_ATOMIC_CSUB : MUBUF_Pseudo_Atomics_RTN <
1050+
defm BUFFER_ATOMIC_CSUB : MUBUF_Pseudo_Atomics <
10511051
"buffer_atomic_csub", VGPR_32, i32, int_amdgcn_global_atomic_csub
10521052
>;
1053-
1054-
let OtherPredicates = [HasAtomicCSubNoRtnInsts] in
1055-
defm BUFFER_ATOMIC_CSUB : MUBUF_Pseudo_Atomics_NO_RTN <
1056-
"buffer_atomic_csub", VGPR_32, i32
1057-
>;
10581053
}
10591054

10601055
let SubtargetPredicate = isGFX8GFX9 in {

llvm/lib/Target/AMDGPU/FLATInstructions.td

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -871,12 +871,8 @@ defm GLOBAL_ATOMIC_DEC_X2 : FLAT_Global_Atomic_Pseudo <"global_atomic_dec_x2",
871871
VReg_64, i64>;
872872

873873
let SubtargetPredicate = HasGFX10_BEncoding in {
874-
defm GLOBAL_ATOMIC_CSUB : FLAT_Global_Atomic_Pseudo_RTN <"global_atomic_csub",
874+
defm GLOBAL_ATOMIC_CSUB : FLAT_Global_Atomic_Pseudo <"global_atomic_csub",
875875
VGPR_32, i32>;
876-
877-
let OtherPredicates = [HasAtomicCSubNoRtnInsts] in
878-
defm GLOBAL_ATOMIC_CSUB : FLAT_Global_Atomic_Pseudo_NO_RTN <"global_atomic_csub",
879-
VGPR_32, i32>;
880876
}
881877

882878
defm GLOBAL_LOAD_LDS_UBYTE : FLAT_Global_Load_LDS_Pseudo <"global_load_lds_ubyte">;

0 commit comments

Comments
 (0)