[Lldb-commits] [lldb] [LLDB] Add allow_var_updates to DIL and CanUpdateVar to SetValueFromInteger (PR #186421)

[via lldb-commits]

cmtice wrote:

> > > Maybe we could add a new option to `StackFrame::ExpressionPathOption` that allows updating values instead of making a new `allow_var_updates` parameter for `GetValueForVariableExpressionPath`? Some the callers already fill these options in, and it seems more in line with other options there, instead of being DIL-specific.
> > 
> > 
> > What do you think about doing this instead of new argument? The `frame var` console command would only have to add another option here:
> > https://github.com/llvm/llvm-project/blob/1d883c675e786920d9e1c6af4fb5fcfb40b4f09a/lldb/source/Commands/CommandObjectFrame.cpp#L610-L613
> 
> Let me give that a try and see how it goes.

Ok, that seems to work well. I have made that change.

https://github.com/llvm/llvm-project/pull/186421