[llvm] r273857 - [sparc] Simplify slow and verbose string matching code to startswith_lower.
Benjamin Kramer via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 27 02:38:56 PDT 2016
Author: d0k
Date: Mon Jun 27 04:38:56 2016
New Revision: 273857
URL: http://llvm.org/viewvc/llvm-project?rev=273857&view=rev
Log:
[sparc] Simplify slow and verbose string matching code to startswith_lower.
No functionality change intended, found by cppcheck. PR28274.
Modified:
llvm/trunk/lib/Target/Sparc/LeonPasses.cpp
Modified: llvm/trunk/lib/Target/Sparc/LeonPasses.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Sparc/LeonPasses.cpp?rev=273857&r1=273856&r2=273857&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Sparc/LeonPasses.cpp (original)
+++ llvm/trunk/lib/Target/Sparc/LeonPasses.cpp Mon Jun 27 04:38:56 2016
@@ -169,10 +169,9 @@ bool FixFSMULD::runOnMachineFunction(Mac
Reg3Index = MI.getOperand(2).getReg();
}
else if (MI.isInlineAsm()) {
- std::string AsmString (MI.getOperand(InlineAsm::MIOp_AsmString).getSymbolName());
- std::string FMULSOpCoode ("fsmuld");
- std::transform(AsmString.begin(), AsmString.end(), AsmString.begin(), ::tolower);
- if (AsmString.find(FMULSOpCoode) == 0) { // this is an inline FSMULD instruction
+ StringRef AsmString(
+ MI.getOperand(InlineAsm::MIOp_AsmString).getSymbolName());
+ if (AsmString.startswith_lower("fsmuld")) {
//errs() << "Detected InlineAsm FSMULD\n";
unsigned StartOp = InlineAsm::MIOp_FirstOperand;
@@ -281,10 +280,9 @@ bool ReplaceFMULS::runOnMachineFunction(
Reg3Index = MI.getOperand(2).getReg();
}
else if (MI.isInlineAsm()) {
- std::string AsmString (MI.getOperand(InlineAsm::MIOp_AsmString).getSymbolName());
- std::string FMULSOpCoode ("fmuls");
- std::transform(AsmString.begin(), AsmString.end(), AsmString.begin(), ::tolower);
- if (AsmString.find(FMULSOpCoode) == 0) { // this is an inline FMULS instruction
+ StringRef AsmString(
+ MI.getOperand(InlineAsm::MIOp_AsmString).getSymbolName());
+ if (AsmString.startswith_lower("fmuls")) {
//errs() << "Detected InlineAsm FMULS\n";
unsigned StartOp = InlineAsm::MIOp_FirstOperand;
@@ -377,15 +375,12 @@ bool FixAllFDIVSQRT::runOnMachineFunctio
unsigned Opcode = MI.getOpcode();
if (MI.isInlineAsm()) {
- std::string AsmString (MI.getOperand(InlineAsm::MIOp_AsmString).getSymbolName());
- std::string FSQRTDOpCode ("fsqrtd");
- std::string FDIVDOpCode ("fdivd");
- std::transform(AsmString.begin(), AsmString.end(), AsmString.begin(), ::tolower);
- if (AsmString.find(FSQRTDOpCode) == 0) { // this is an inline fsqrts instruction
+ StringRef AsmString(
+ MI.getOperand(InlineAsm::MIOp_AsmString).getSymbolName());
+ if (AsmString.startswith_lower("fsqrtd")) {
//errs() << "Detected InlineAsm FSQRTD\n";
Opcode = SP::FSQRTD;
- }
- else if (AsmString.find(FDIVDOpCode) == 0) { // this is an inline fsqrts instruction
+ } else if (AsmString.startswith_lower("fdivd")) {
//errs() << "Detected InlineAsm FDIVD\n";
Opcode = SP::FDIVD;
}
More information about the llvm-commits
mailing list