[Lldb-commits] [lldb] [lldb] Change the implementation of Status to store an llvm::Error (NFC) (PR #106774)

[Pavel Labath via lldb-commits]

https://github.com/labath commented:

> Status is already extensively unittested.

The fact that you were able to pull this switch off without changing a single line of test code tells me otherwise. Surely there is *some* new functionality being added here. 

>  Is there something you feel is missing?

For example, the value of GetError()/GetType() for multi-error objects. And the ability to copy an ErrorInfo object and make sure that the copy preserves its type and content. Maybe there's more, but these are the most obvious new/changed behaviors of the class.

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