[Lldb-commits] [lldb] Make result variables obey their dynamic values in subsequent expressions (PR #168611)

[via lldb-commits]

================
@@ -250,8 +250,14 @@ LLVMUserExpression::DoExecute(DiagnosticManager &diagnostic_manager,
     }
   }
 
-  if (FinalizeJITExecution(diagnostic_manager, exe_ctx, result,
+  if (FinalizeJITExecution(diagnostic_manager, exe_ctx, result_sp,
                            function_stack_bottom, function_stack_top)) {
+    //    if (result_sp) {
+    //      // This is a bit of a hack, replace with a real API.  Trying to
+    //      force
+    //      // fetching all the dynamic info at this point.
+    //      result_sp->GetValueObject();
+    //    }
----------------
jimingham wrote:

Yes, I thought at some point that I needed to force this here, but it was causing other problems and I couldn't figure out a test case where it mattered that I do this.  So it should just go.

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