Hi,
On 04/11/16 10:58, James Cowgill wrote:
> > + #include <cstdinti>
> Typo? You want <cstdint>
Yes, this is my fault. The name of the library is not properly specified.
> > #include <atomic>
> > + std::atomic<uintptr_t> x;
> > + std::atomic<uintmax_t> y;
> I don't think you should rely on these types being in the global namespace.
I modified test and removed uintptr_t type, because this type may not be
defined in some library implementations.
More about that:
http://www.cplusplus.com/reference/cstdint/
Could you please take a look at new patch I attached? It includes a new test
for checking is libatomic needed to link.
Regards,
Radovan
--- drumgizmo-0.9.11.orig/configure.ac
+++ drumgizmo-0.9.11/configure.ac
@@ -473,9 +473,10 @@ dnl ======================
AC_MSG_CHECKING([for the need for linkage with libatomic])
AC_LANG_PUSH([C++])
AC_LINK_IFELSE([AC_LANG_SOURCE[
+ #include <cstdint>
#include <atomic>
int main() {
- struct Test { int val; };
+ struct Test { uintmax_t val; };
std::atomic<Test> s;
return s.is_lock_free()?1:0;
}
