[Lldb-commits] [lldb] [LLDB][SBSaveCore] Implement a selectable threadlist for Core Options. (PR #100443)

[Jacob Lalonde via lldb-commits]

Jlalond wrote:

> Stepping back a bit, do we gain anything in `SaveCoreOptions` by having `m_process_sp` be optional? Is there a distinction between an empty ProcessSP and no ProcessSP object?

No, this was mostly a mistake on my part. I wanted to describe process as optional but required for advanced functions. Making it optional gave us no value over nullability.

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