[Lldb-commits] [lldb] [lldb/Target] Add GetStartSymbol method to DynamicLoader plugins (PR #99673)

[Med Ismail Bennani via lldb-commits]

================
@@ -329,6 +330,11 @@ class DynamicLoader : public PluginInterface {
   /// safe to call certain APIs or SPIs.
   virtual bool IsFullyInitialized() { return true; }
 
+  /// Return the `start` function \b symbol in the dynamic loader module.
+  virtual llvm::Expected<lldb_private::Symbol> GetStartSymbol() {
----------------
medismailben wrote:

In my understanding, every dynamic loader should have an equivalent to a start symbol, that's why I went with `Expected` instead of `optional`. I believe that long term, each dynamic loader plugin would have to implement this method for their own platform and only the base class would have the `UnimplementedError`.

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