[Lldb-commits] [PATCH] D72510: [lldb/Expression] Improve interpreter error message with a non-running target

[Med Ismail Bennani via Phabricator via lldb-commits]

mib updated this revision to Diff 237911.
mib retitled this revision from "[lldb/Expression] Improve interpreter error message  with a non-running target" to "[lldb/Expression] Improve interpreter error message with a non-running target".
mib edited the summary of this revision.
mib added a comment.

Moved error to ClangExpressionParser instead of IRInterpreter.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72510

Files:
  lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp


Index: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp
===================================================================
--- lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp
+++ lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp
@@ -1258,8 +1258,10 @@
           interpret_error, interpret_function_calls);
 
       if (!can_interpret && execution_policy == eExecutionPolicyNever) {
-        err.SetErrorStringWithFormat("Can't run the expression locally: %s",
-                                     interpret_error.AsCString());
+        err.SetErrorStringWithFormat(
+            "Can't interpret the expression without a running target. "
+            "Interpretation failed due to: %s",
+            interpret_error.AsCString());
         return err;
       }
     }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72510.237911.patch
Type: text/x-patch
Size: 831 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200114/02e6c40d/attachment.bin>