================
@@ -56,8 +58,10 @@ class SaveCoreOptions {
   std::optional<lldb_private::FileSpec> m_file;
   std::optional<lldb::SaveCoreStyle> m_style;
   lldb::ProcessSP m_process_sp;
-  std::unordered_set<lldb::tid_t> m_threads_to_save;
+  std::unordered_map<lldb::tid_t, lldb::ThreadSP> m_threads_to_save;
----------------
clayborg wrote:

Not sure if we need this complexity of an unordered_map + vector. Most users 
will add a couple of threads, or none at all. Can we just change this to be:
```
std::vector<lldb::tid_t> m_threads_to_save;
```
The get rid of `m_thread_indexes`?

https://github.com/llvm/llvm-project/pull/122541
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to