Right now LLDB_API is used only for windows and turns into __declspec(dllexport) or __declspec(dllimport). The main question is if we are going to try and use these for other compiles (like using the GCC attribute for visibility). If we ever decide to use the GCC attributes in LLDB_API, then we might need one for functions: LLDB_API_FUNCTION and one for classes LLDB_API_CLASS.
So for now this is fine as the __declspec() works on classes for windows, but if we put __attribute__((....)) in there I don't believe this works on class definitions to export the entire class... So yes, OK to apply. On Mar 12, 2014, at 9:08 AM, Abid, Hafiz <[email protected]> wrote: > Hi, > This patch add LLDB_API to the class declaration for the API classes. It was > already present in the forward declaration in SBDefines.h. I need LLDB_API to > be part of the declaration to build a DLL that exports the API and can be > successfully linked with driver. This only effects Windows. Ok to apply? > > Regards, > Abid > <api.patch>_______________________________________________ > lldb-dev mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev _______________________________________________ lldb-dev mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
