diff options
author | Fangrui Song <i@maskray.me> | 2023-09-27 13:27:51 -0700 |
---|---|---|
committer | Fangrui Song <i@maskray.me> | 2023-09-27 13:27:51 -0700 |
commit | d87f9e287217053bd4613e025c3fb0a941fd0196 (patch) | |
tree | 3abf24136ce8e0803b2ae91496f8ceb11be9e223 /llvm/tools/llvm-dwarfdump | |
parent | 61b0f12d6b9cdcb6bb3dd679e3a3c36fa94daeae (diff) | |
download | llvm-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.cpp | 18 |
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()); } |