Magic-Mirror/llvm-capstone
1
0
Fork 0
mirror of https://github.com/capstone-engine/llvm-capstone.git synced 2025-02-17 08:21:13 +00:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

[lldb] Remove CompileUnit::SetSupportFiles overload (NFC)

Browse Source 
CompileUnit::SetSupportFiles had two overloads, one that took and lvalue
reference and one that takes an rvalue reference. This removes both and
replaces it with an overload that takes the FileSpecList by value and
moves it into the member variable.

Because we're storing the value as a member, this covers both cases. If
the new FileSpecList was passed by lvalue reference, we'd copy it into
the member anyway. If it was passed as an rvalue reference, we'll have
created a new instance using its move and then immediately move it again
into our member. In either case the number of copies remains unchanged.
This commit is contained in:
Jonas Devlieghere 2023-10-19 11:09:26 -07:00
parent 5175cd777c
commit 5bae3a0b0c
No known key found for this signature in database
GPG Key ID: 49CC0BD90FDEED4D
2 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lldb/include/lldb/Symbol/CompileUnit.h
View File

@ -331,8 +331,7 @@ public:
/// A line table object pointer that this object now owns.
void SetLineTable(LineTable *line_table);
void SetSupportFiles(const FileSpecList &support_files);
void SetSupportFiles(FileSpecList &&support_files);
void SetSupportFiles(FileSpecList support_files);
void SetDebugMacros(const DebugMacrosSP &debug_macros);

6
lldb/source/Symbol/CompileUnit.cpp
View File

@ -178,11 +178,7 @@ void CompileUnit::SetLineTable(LineTable *line_table) {
m_line_table_up.reset(line_table);
}
void CompileUnit::SetSupportFiles(const FileSpecList &support_files) {
m_support_files = support_files;
}
void CompileUnit::SetSupportFiles(FileSpecList &&support_files) {
void CompileUnit::SetSupportFiles(FileSpecList support_files) {
m_support_files = std::move(support_files);
}