[Lldb-commits] [PATCH] D30984: Centralize libc++ test skipping logic

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Mar 29 13:24:36 PDT 2017

labath added a comment.

In https://reviews.llvm.org/D30984#711806, @EricWF wrote:

> I don't see anything wrong with this, but I only know libc++ and not LLDB.

I'm not sure why you ended up here. I think you have too wide phabricator filter somewhere. :)

>>   libc++'s atomic does not play well with gcc on linux
> It should... Can you elaborate on this issue? I suspect this may be a libc++ bug.

It checks for `#if __has_feature(cxx_atomic)` and then aborts if it is not found. However, this is in the system libc++ on ubuntu (lldb uses system libc++ for reasons which are too long to go into right now), which is quite old. I don't see a similar check in the current trunk, so it's possible that has been fixed since then.


More information about the lldb-commits mailing list