[Lldb-commits] [PATCH] D97249: [lldb] Support debugging utility functions

[Shafik Yaghmour via Phabricator via lldb-commits]

shafik added inline comments.


================
Comment at: lldb/source/Expression/FunctionCaller.cpp:324
   EvaluateExpressionOptions real_options = options;
   real_options.SetDebug(false);
+  real_options.SetGenerateDebugInfo(debug);
----------------
It feels a little weird you are using the name `debug` but using it in the call to `SetDebug(...)` but are using it in the call to `SetGenerateDebugInfo(...)`


================
Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionSourceCode.cpp:38
 const char *ClangExpressionSourceCode::g_expression_prefix =
-"#line 1 \"" PREFIX_NAME R"("
+    "#line 1 \"" PREFIX_NAME R"("#line 1
 #ifndef offsetof
----------------
Why additional `#line 1`.


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

https://reviews.llvm.org/D97249