Skip to content

Commit eff28bd

Browse files
[AMDGPU] Use StringRef::consume_back (NFC) (#146194)
Note that StringRef::consume_back returns true while consuming the given prefix if present.
1 parent c73906e commit eff28bd

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6451,26 +6451,26 @@ StringRef AMDGPUAsmParser::parseMnemonicSuffix(StringRef Name) {
64516451
setForcedDPP(false);
64526452
setForcedSDWA(false);
64536453

6454-
if (Name.ends_with("_e64_dpp")) {
6454+
if (Name.consume_back("_e64_dpp")) {
64556455
setForcedDPP(true);
64566456
setForcedEncodingSize(64);
6457-
return Name.substr(0, Name.size() - 8);
6457+
return Name;
64586458
}
6459-
if (Name.ends_with("_e64")) {
6459+
if (Name.consume_back("_e64")) {
64606460
setForcedEncodingSize(64);
6461-
return Name.substr(0, Name.size() - 4);
6461+
return Name;
64626462
}
6463-
if (Name.ends_with("_e32")) {
6463+
if (Name.consume_back("_e32")) {
64646464
setForcedEncodingSize(32);
6465-
return Name.substr(0, Name.size() - 4);
6465+
return Name;
64666466
}
6467-
if (Name.ends_with("_dpp")) {
6467+
if (Name.consume_back("_dpp")) {
64686468
setForcedDPP(true);
6469-
return Name.substr(0, Name.size() - 4);
6469+
return Name;
64706470
}
6471-
if (Name.ends_with("_sdwa")) {
6471+
if (Name.consume_back("_sdwa")) {
64726472
setForcedSDWA(true);
6473-
return Name.substr(0, Name.size() - 5);
6473+
return Name;
64746474
}
64756475
return Name;
64766476
}

0 commit comments

Comments
 (0)