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

Alex Langford via lldb-commits lldb-commits at lists.llvm.org
Fri Aug 30 14:12:32 PDT 2024


================
@@ -97,7 +97,7 @@ class LLDB_API SBError {
   friend class lldb_private::ScriptInterpreter;
   friend class lldb_private::python::SWIGBridge;
 
-  SBError(const lldb_private::Status &error);
+  SBError(lldb_private::Status &&error);
----------------
bulbazord wrote:

I have no concerns about Unix-ish platforms, but Windows may be a problem. I know that MSVC's mangling scheme includes access level (private/protected/public), which makes me think that private/protected symbols may be difficult to change while keeping ABI stable on MSVC.
Certainly SBAPI users and libraries shouldn't even be able to use this symbol, but I don't know if Windows will be unhappy that a private symbol changed.

@compnerd Do you have any insight here?

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


More information about the lldb-commits mailing list