[Lldb-commits] [PATCH] D149719: [LLDB] Add a hook to notify REPLs that an expression was evaluated

walter erquinigo via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu May 4 12:34:26 PDT 2023


wallace added inline comments.


================
Comment at: lldb/include/lldb/Expression/REPL.h:124
+                        const lldb::ValueObjectSP &result_valobj_sp,
+                        const Status &error) {
+    return llvm::Error::success();
----------------
bulbazord wrote:
> Do you still need the `Status` parameter?
that status is the result of the expression evaluation, so it's useful to have it here


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149719/new/

https://reviews.llvm.org/D149719



More information about the lldb-commits mailing list