[Lldb-commits] [lldb] [lldb] Add target groups for related targets (PR #229180)

[Greg Clayton via lldb-commits]

================
@@ -411,6 +416,43 @@ void Target::Destroy() {
   ClearDummySignals(signal_args);
 }
 
+std::vector<TargetGroupSP> Target::GetTargetGroups() const {
+  const auto &mutex = m_debugger.GetTargetGroupList().m_mutex;
+  std::lock_guard<std::recursive_mutex> guard(*mutex);
+  std::vector<TargetGroupSP> groups;
----------------
clayborg wrote:

Anytime you see a pattern like this where you grab a member variable from another class and then use it usually means this code belongs as a method in that class. See comment below.

https://github.com/llvm/llvm-project/pull/229180