[PATCH] D16614: Use "long long int" when checking whether atomics are supported.

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 17:21:32 PST 2016


jyknight accepted this revision.
jyknight added a comment.
This revision is now accepted and ready to land.

In http://reviews.llvm.org/D16614#336817, @jroelofs wrote:

> What happens on platforms that don't need the lib to make `std::atomic<long long int> x;` work?


Then the first check succeeds, and they don't get -latomic added to the link line (HAVE_CXX_ATOMICS_WITHOUT_LIB gets set)


http://reviews.llvm.org/D16614





More information about the llvm-commits mailing list