================
@@ -1012,33 +1006,26 @@ bool SBFrame::GetDescription(SBStream &description) {
SBValue SBFrame::EvaluateExpression(const char *expr) {
LLDB_INSTRUMENT_VA(this, expr);
- SBValue result;
std::unique_lock<std::recursive_mutex> lock;
ExecutionContext exe_ctx(m_opaque_sp.get(), lock);
StackFrame *frame = exe_ctx.GetFramePtr();
Target *target = exe_ctx.GetTargetPtr();
+ SBExpressionOptions options;
if (frame && target) {
----------------
JDevlieghere wrote:
Maybe? I'm not a 100% sure which is why I wasn't suggesting it. Even the
StackFrame implementation itself seems pretty inconsistent in whether it uses
the execution context or `CalculateTarget`. I'd keep the existing behavior to
avoid regressing anything.
https://github.com/llvm/llvm-project/pull/119103
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
