aboutsummaryrefslogtreecommitdiff
path: root/llvm/tools/llvm-dwarfdump
diff options
context:
space:
mode:
authorFangrui Song <i@maskray.me>2023-09-27 13:27:51 -0700
committerFangrui Song <i@maskray.me>2023-09-27 13:27:51 -0700
commitd87f9e287217053bd4613e025c3fb0a941fd0196 (patch)
tree3abf24136ce8e0803b2ae91496f8ceb11be9e223 /llvm/tools/llvm-dwarfdump
parent61b0f12d6b9cdcb6bb3dd679e3a3c36fa94daeae (diff)
downloadllvm-d87f9e287217053bd4613e025c3fb0a941fd0196.zip
llvm-d87f9e287217053bd4613e025c3fb0a941fd0196.tar.gz
llvm-d87f9e287217053bd4613e025c3fb0a941fd0196.tar.bz2
llvm/tools: Fix some performance-for-range-copy issues. NFC
Inspired by https://reviews.llvm.org/D139487 , but I apply manual fixes when clang-tidy does not provide the best fix.
Diffstat (limited to 'llvm/tools/llvm-dwarfdump')
-rw-r--r--llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp18
1 files changed, 9 insertions, 9 deletions
diff --git a/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp b/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
index 156e10c..8180ad2 100644
--- a/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
+++ b/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
@@ -337,10 +337,10 @@ static bool filterArch(ObjectFile &Obj) {
return true;
if (auto *MachO = dyn_cast<MachOObjectFile>(&Obj)) {
- for (auto Arch : ArchFilters) {
+ for (const StringRef Arch : ArchFilters) {
// Match architecture number.
unsigned Value;
- if (!StringRef(Arch).getAsInteger(0, Value))
+ if (!Arch.getAsInteger(0, Value))
if (Value == getCPUType(*MachO))
return true;
@@ -649,7 +649,7 @@ static bool dumpObjectFile(ObjectFile &Obj, DWARFContext &DICtx,
// Handle the --name option.
if (!Name.empty()) {
StringSet<> Names;
- for (auto name : Name)
+ for (const auto &name : Name)
Names.insert((IgnoreCase && !UseRegex) ? StringRef(name).lower() : name);
filterByName(Names, DICtx.normal_units(), OS, GetRegName);
@@ -698,7 +698,7 @@ static bool handleArchive(StringRef Filename, Archive &Arch,
HandlerFn HandleObj, raw_ostream &OS) {
bool Result = true;
Error Err = Error::success();
- for (auto Child : Arch.children(Err)) {
+ for (const auto &Child : Arch.children(Err)) {
auto BuffOrErr = Child.getMemoryBufferRef();
error(Filename, BuffOrErr.takeError());
auto NameOrErr = Child.getName();
@@ -848,19 +848,19 @@ int main(int argc, char **argv) {
bool Success = true;
if (Verify) {
- for (auto Object : Objects)
+ for (StringRef Object : Objects)
Success &= handleFile(Object, verifyObjectFile, OutputFile.os());
} else if (Statistics) {
- for (auto Object : Objects)
+ for (StringRef Object : Objects)
Success &= handleFile(Object, collectStatsForObjectFile, OutputFile.os());
} else if (ShowSectionSizes) {
- for (auto Object : Objects)
+ for (StringRef Object : Objects)
Success &= handleFile(Object, collectObjectSectionSizes, OutputFile.os());
} else if (ShowSources) {
- for (auto Object : Objects)
+ for (StringRef Object : Objects)
Success &= handleFile(Object, collectObjectSources, OutputFile.os());
} else {
- for (auto Object : Objects)
+ for (StringRef Object : Objects)
Success &= handleFile(Object, dumpObjectFile, OutputFile.os());
}