================
@@ -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
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to