https://github.com/jasonmolenda approved this pull request.

LGTM.  I'm fine with changing the default to show the current cached unwind 
plans, this command is only used by lldb maintainers (mostly, me), normal users 
won't notice a change here.  Jonas' suggestion of moving this command under 
`target modules dump` would make sense to me too, but you shouldn't need to do 
that in this PR unless you want to, I know there's a lot of mechanical code 
moving to implement a change like that.  Again this is not a normal user 
command, I don't mind changing its behavior or name if anyone thinks that is 
better.

https://github.com/llvm/llvm-project/pull/125839
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to