Author: Dave Lee Date: 2026-05-31T09:54:09-07:00 New Revision: c42f1884730fb51a5f802ad621c0cc7f68715424
URL: https://github.com/llvm/llvm-project/commit/c42f1884730fb51a5f802ad621c0cc7f68715424 DIFF: https://github.com/llvm/llvm-project/commit/c42f1884730fb51a5f802ad621c0cc7f68715424.diff LOG: [lldb] Fix up comment placement (NFC) (#200671) Follow up to https://github.com/llvm/llvm-project/pull/200084 Added: Modified: lldb/source/Commands/CommandObjectFrame.cpp Removed: ################################################################################ diff --git a/lldb/source/Commands/CommandObjectFrame.cpp b/lldb/source/Commands/CommandObjectFrame.cpp index 742f7026464f0..5c79d758bafd7 100644 --- a/lldb/source/Commands/CommandObjectFrame.cpp +++ b/lldb/source/Commands/CommandObjectFrame.cpp @@ -731,6 +731,10 @@ may even involve JITing and running code in the target program.)"); valobj_sp = frame->GetValueForVariableExpressionPath( entry.ref(), m_varobj_options.use_dynamic, expr_path_options, var_sp, error); + // Check only the `error` argument, because doing + // `valobj_sp->GetError()` will update the value and potentially + // return a new error that happens during the update, even if + // `GetValueForVariableExpressionPath` reported no errors. if (valobj_sp && error.Success()) { result.GetValueObjectList().Append(valobj_sp); @@ -770,10 +774,6 @@ may even involve JITing and running code in the target program.)"); break; } if (!found_recognized) { - // Check only the `error` argument, because doing - // `valobj_sp->GetError()` will update the value and potentially - // return a new error that happens during the update, even if - // `GetValueForVariableExpressionPath` reported no errors. if (error.Fail()) result.SetError(error.takeError()); else _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
