[Lldb-commits] [lldb] [lldb] Enable chaining multiple scripted frame providers per thread (PR #172849)

[Alex Langford via lldb-commits]

================
@@ -1409,8 +1409,8 @@ class Thread : public std::enable_shared_from_this<Thread>,
   /// The Thread backed by this thread, if any.
   lldb::ThreadWP m_backed_thread;
 
-  /// The Scripted Frame Provider, if any.
-  lldb::SyntheticFrameProviderSP m_frame_provider_sp;
+  /// The Scripted Frame Providers for this thread.
+  std::vector<lldb::SyntheticFrameProviderSP> m_frame_providers;
----------------
bulbazord wrote:

Suggestion: name this something like "m_executed_frame_providers" or "m_loaded_frame_providers". From the implementation, I think that this is the list of providers that have *already run*. But I initially guessed/assumed that this might already be populated when the SyntheticStackFrameList was created.

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