jimingham wrote: The way we've been doing the `po` implementations for a while now is: "If the object description succeeds, print the object description, otherwise print the value." This routine gets used here:
llvm::Expected<std::string> object_desc = (value_printed || summary_printed) ? GetMostSpecializedValue().GetObjectDescription() : GetDescriptionForDisplay(); So it's supposed to handle the case where we haven't yet printed any value or summary for the `po` result. By returning if you see an expression error, you're changing this UI to NOT print the value if the `po` expression fails for some reason. Is that your intent? https://github.com/llvm/llvm-project/pull/152417 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits