Here is a patch to use std::atomic. Removed RefCounter.h/cpp since not used anymore as well (it would need atomic anyway).
If it looks good to you, I will commit asap to fix the build. Sorry again that I didn't know LLVM headers could not be used from LLDB headers. On Thu, Sep 5, 2013 at 6:40 PM, Virgile Bello <[email protected]>wrote: > I see, sorry about that. > I will fix it by adding the atomic stuff in lldb/Host then. > > > On Thu, Sep 5, 2013 at 8:56 AM, <[email protected]> wrote: > >> Yes, I agree with this part. It should not be necessary to have the LLVM >> headers around to build programs that only use the lldb SB API's. >> >> Jim >> >> On Sep 4, 2013, at 4:36 PM, Malea, Daniel <[email protected]> wrote: >> >> > This commit introduces a dependency in the LLDB API on the LLVM headers, >> > which is not ideal because users of the LLDB API shouldn't be required >> to >> > have LLVM headers around IMHO. >> > >> >> >
lldb-atomic.patch
Description: Binary data
_______________________________________________ lldb-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
