[llvm] [llvm] Use StringRef::{starts_with,find} (NFC) (PR #139661)
via llvm-commits
llvm-commits at lists.llvm.org
Mon May 12 19:47:01 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-debuginfo
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
Calling find/contains in the StringRef domain allows us to avoid
creating temporary instances of std::string.
---
Full diff: https://github.com/llvm/llvm-project/pull/139661.diff
2 Files Affected:
- (modified) llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp (+1-1)
- (modified) llvm/lib/TableGen/Record.cpp (+4-4)
``````````diff
diff --git a/llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp b/llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp
index 22137ea172240..7a0256f10ea60 100644
--- a/llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp
+++ b/llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp
@@ -780,7 +780,7 @@ llvm::Error DwarfTransformer::verify(StringRef GsymPath,
const auto &dii = DwarfInlineInfos.getFrame(Idx);
gsymFilename = LR->getSourceFile(Idx);
// Verify function name
- if (dii.FunctionName.find(gii.Name.str()) != 0)
+ if (!StringRef(dii.FunctionName).starts_with(gii.Name))
Out << "error: address " << HEX64(Addr) << " DWARF function \""
<< dii.FunctionName.c_str()
<< "\" doesn't match GSYM function \"" << gii.Name << "\"\n";
diff --git a/llvm/lib/TableGen/Record.cpp b/llvm/lib/TableGen/Record.cpp
index 51ed259132497..0d9fcb0e63dae 100644
--- a/llvm/lib/TableGen/Record.cpp
+++ b/llvm/lib/TableGen/Record.cpp
@@ -1796,11 +1796,11 @@ const Init *TernOpInit::Fold(const Record *CurRec) const {
if (LHSs && MHSs && RHSs) {
std::string Val = RHSs->getValue().str();
- std::string::size_type found;
- std::string::size_type idx = 0;
+ StringRef::size_type found;
+ StringRef::size_type idx = 0;
while (true) {
- found = Val.find(LHSs->getValue().str(), idx);
- if (found == std::string::npos)
+ found = StringRef(Val).find(LHSs->getValue(), idx);
+ if (found == StringRef::npos)
break;
Val.replace(found, LHSs->getValue().size(), MHSs->getValue().str());
idx = found + MHSs->getValue().size();
``````````
</details>
https://github.com/llvm/llvm-project/pull/139661
More information about the llvm-commits
mailing list