[Lldb-commits] [lldb] [LLDB] Support enabling/disabling InstrumentationRuntime plugins during a debug session (PR #190083)

[Jonas Devlieghere via lldb-commits]

================
@@ -92,6 +95,18 @@ class InstrumentationRuntime
 
   bool IsActive() const { return m_is_active; }
 
+  /// Called when the user runs
+  /// `plugin enable instrumentation-runtime.<name>`
+  ///
+  /// \returns true iff on success
+  virtual bool Enable();
----------------
JDevlieghere wrote:

Would it be more future proof to have this return an `llvm::Error`? Even if it always return `Error::success()` for now.

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