diff --git a/llvm/lib/Target/AMDGPU/DSInstructions.td b/llvm/lib/Target/AMDGPU/DSInstructions.td --- a/llvm/lib/Target/AMDGPU/DSInstructions.td +++ b/llvm/lib/Target/AMDGPU/DSInstructions.td @@ -842,9 +842,6 @@ foreach vt = VReg_64.RegTypes in { defm : DSReadPat_mc ; -} - -foreach vt = VReg_64.RegTypes in { defm : DSWritePat_mc ; } @@ -852,34 +849,25 @@ foreach vt = VReg_96.RegTypes in { defm : DSReadPat_mc ; -} - -foreach vt = VReg_96.RegTypes in { defm : DSWritePat_mc ; } foreach vt = VReg_128.RegTypes in { defm : DSReadPat_mc ; -} - -foreach vt = VReg_128.RegTypes in { defm : DSWritePat_mc ; } let SubtargetPredicate = HasUnalignedAccessMode in { -// FIXME: Is ds_read_b96/ds_write_b96 better choice in unaligned-access-mode? +// FIXME: From performance point of view, is ds_read_b96/ds_write_b96 better choice +// for unaligned accesses? foreach vt = VReg_96.RegTypes in { defm : DSReadPat_mc ; -} - -foreach vt = VReg_96.RegTypes in { defm : DSWritePat_mc ; } -// For performance reasons, *do not* select ds_read_b128/ds_write_b128 in -// unaligned-access-mode. At lower alignments ds_read2_b64/ds_write2_b64 is -// always a better choice. +// For performance reasons, *do not* select ds_read_b128/ds_write_b128 for unaligned +// accesses. } // End SubtargetPredicate = HasUnalignedAccessMode