https://github.com/xgupta updated
https://github.com/llvm/llvm-project/pull/94840
>From 25fb87d4bb4af55e642ec386f104412b39065a83 Mon Sep 17 00:00:00 2001
From: Shivam Gupta <shivam98....@gmail.com>
Date: Sat, 8 Jun 2024 10:43:01 +0530
Subject: [PATCH] [lldb] Use const reference for range variables to improve
performance (NFC)
Cppcheck recommends using a const reference for range variables in a for-each
loop.
This avoids unnecessary copying of elements, improving performance.
Caught by cppcheck -
lldb/source/API/SBBreakpoint.cpp:717:22: performance: Range variable 'name'
should be declared as const reference. [iterateByValue]
lldb/source/API/SBTarget.cpp:1150:15: performance: Range variable 'name' should
be declared as const reference. [iterateByValue]
lldb/source/Breakpoint/Breakpoint.cpp:888:26: performance: Range variable
'name' should be declared as const reference. [iterateByValue]
Fix #91213
Fix #91217
Fix #91219
---
lldb/source/API/SBBreakpoint.cpp | 2 +-
lldb/source/API/SBTarget.cpp | 2 +-
lldb/source/Breakpoint/Breakpoint.cpp | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/lldb/source/API/SBBreakpoint.cpp b/lldb/source/API/SBBreakpoint.cpp
index f1fb6f904f5f0..3d908047f9455 100644
--- a/lldb/source/API/SBBreakpoint.cpp
+++ b/lldb/source/API/SBBreakpoint.cpp
@@ -714,7 +714,7 @@ void SBBreakpoint::GetNames(SBStringList &names) {
bkpt_sp->GetTarget().GetAPIMutex());
std::vector<std::string> names_vec;
bkpt_sp->GetNames(names_vec);
- for (std::string name : names_vec) {
+ for (const std::string &name : names_vec) {
names.AppendString(name.c_str());
}
}
diff --git a/lldb/source/API/SBTarget.cpp b/lldb/source/API/SBTarget.cpp
index 962ce9ba83cc7..adb9e645610ba 100644
--- a/lldb/source/API/SBTarget.cpp
+++ b/lldb/source/API/SBTarget.cpp
@@ -1147,7 +1147,7 @@ void SBTarget::GetBreakpointNames(SBStringList &names) {
std::vector<std::string> name_vec;
target_sp->GetBreakpointNames(name_vec);
- for (auto name : name_vec)
+ for (const auto &name : name_vec)
names.AppendString(name.c_str());
}
}
diff --git a/lldb/source/Breakpoint/Breakpoint.cpp
b/lldb/source/Breakpoint/Breakpoint.cpp
index ae845e92762b9..dc80d435ad444 100644
--- a/lldb/source/Breakpoint/Breakpoint.cpp
+++ b/lldb/source/Breakpoint/Breakpoint.cpp
@@ -885,7 +885,7 @@ void Breakpoint::GetDescription(Stream *s,
lldb::DescriptionLevel level,
s->Printf("Names:");
s->EOL();
s->IndentMore();
- for (std::string name : m_name_list) {
+ for (const std::string &name : m_name_list) {
s->Indent();
s->Printf("%s\n", name.c_str());
}
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
