Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
You need to use LIBLLDB_API instead of __declspec(export). Otherwise
someone linking against liblldb and trying to call this won't work
correctly. Check the annotation given to the classes in the SB headers for
the exact name of the macro.
Comment Actions
Sorry! I'll fix this as soon as I get home.
I have a question though, isn't this only a problem if whatever's linking
against lldb includes this header? This isn't in the API, so that shouldn't
happen. I also don't know if it's correct to label it LLDB_API when it's
not actually part of the API.
Comment Actions
True that it only needs to be included if it's part of the api. Are we
sure it can't happen through a transitive include?
As long as the user of liblldb doesn't see the declaration it shouldn't be
a problem.