================
@@ -38,7 +65,10 @@ class FileSpecList {
FileSpecList(FileSpecList &&rhs) = default;
/// Initialize this object from a vector of FileSpecs
- FileSpecList(std::vector<FileSpec> &&rhs) : m_files(std::move(rhs)) {}
+ FileSpecList(std::vector<FileSpec> &&rhs) {
+ for (auto &fs : rhs)
+ m_files.emplace_back(fs);
----------------
adrian-prantl wrote:
Do you have a suggestion for how to fix this? Would adding a move constructor
to FileSpec address your concern?
Note that FileSpecList is a value type that gets copied a lot, which informs
many of the odd design choices in this patch.
https://github.com/llvm/llvm-project/pull/75880
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
