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

Vasileios Kalintiris via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 16:45:58 PST 2016


vkalintiris created this revision.
vkalintiris added reviewers: dsanders, chandlerc, jyknight.
vkalintiris added a subscriber: llvm-commits.

Some targets implement 4-byte atomic related functions without calling
into libatomic, while for 8-byte atomics they need to make a function
call. By using a long long int we can identify such cases.

http://reviews.llvm.org/D16614

Files:
  cmake/modules/CheckAtomic.cmake

Index: cmake/modules/CheckAtomic.cmake
===================================================================
--- cmake/modules/CheckAtomic.cmake
+++ cmake/modules/CheckAtomic.cmake
@@ -10,7 +10,7 @@
   set(CMAKE_REQUIRED_FLAGS "-std=c++11")
   CHECK_CXX_SOURCE_COMPILES("
 #include <atomic>
-std::atomic<int> x;
+std::atomic<long long int> x;
 int main() {
   return x;
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16614.46081.patch
Type: text/x-patch
Size: 371 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160127/f83a42b4/attachment.bin>


More information about the llvm-commits mailing list