[Lldb-commits] [lldb] [lldb] Refactor PrepareToExecuteJITExpression (PR #224888)

[Jonas Devlieghere via lldb-commits]

================
@@ -111,6 +111,23 @@ class LLVMUserExpression : public UserExpression {
                                        ///to the expression have been
                                        ///materialized.
   Materializer::DematerializerSP m_dematerializer_sp; ///< The dematerializer.
+
+private:
+  // Allocate the interpreter's private, host-only stack, if one has not been
+  // allocated already. Idempotent.
+  bool AllocateInterpreterStackFrame(DiagnosticManager &diagnostic_manager,
+                                     Target &target, Process *process);
+
+  // Prepare the struct holding the expression arguments.
+  //
+  // Materialize expression arguments into the argument struct
+  // (m_materialized_address).
+  //
+  // This method allocates the struct on first call, and reuses the original
+  // allocation on subsequent calls.
----------------
JDevlieghere wrote:

```suggestion
  /// Prepare the struct holding the expression arguments.
  ///
  /// Materialize expression arguments into the argument struct
  /// (m_materialized_address).
  ///
  /// This method allocates the struct on first call, and reuses the original
  /// allocation on subsequent calls.
```

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