Fix atomic timestamp builds on Windows and 32-bit GCC

Author: gmartin82

cmake/platforms/linux.cmake

@@ -7,6 +7,9 @@ set(ZP_PLATFORM_SOURCE_FILES
     "${PROJECT_SOURCE_DIR}/src/link/transport/tcp/tcp_posix.c"
     "${PROJECT_SOURCE_DIR}/src/link/transport/udp/udp_posix.c"
     "${PROJECT_SOURCE_DIR}/src/link/transport/serial/tty_posix.c")
+if(CMAKE_C_COMPILER_ID STREQUAL "GNU")
+  set(ZP_PLATFORM_LINK_LIBRARIES atomic)
+endif()
 if(ZP_UDP_MULTICAST_ENABLED)
   list(APPEND ZP_PLATFORM_SOURCE_FILES
        "${PROJECT_SOURCE_DIR}/src/link/transport/udp/udp_multicast_posix.c")

include/zenoh-pico/collections/atomic_impl.h

@@ -57,12 +57,18 @@ static inline memory_order _z_atomic_memory_order(_z_memory_order_t order) {
     }                                                                                                                  \
     bool _z_atomic_##name##_compare_exchange_strong(_z_atomic_##name##_t *var, type *expected, type desired,           \
                                                     _z_memory_order_t success, _z_memory_order_t failure) {            \
+        /* MSVC's C11 atomic macro does not mark memory order parameters as used. */                                   \
+        (void)success;                                                                                                 \
+        (void)failure;                                                                                                 \
         return atomic_compare_exchange_strong_explicit((_z_atomic_##name##_atomic_t *)&var->_value, expected, desired, \
                                                        _z_atomic_memory_order(success),                                \
                                                        _z_atomic_memory_order(failure));                               \
     }                                                                                                                  \
     bool _z_atomic_##name##_compare_exchange_weak(_z_atomic_##name##_t *var, type *expected, type desired,             \
                                                   _z_memory_order_t success, _z_memory_order_t failure) {              \
+        /* MSVC's C11 atomic macro does not mark memory order parameters as used. */                                   \
+        (void)success;                                                                                                 \
+        (void)failure;                                                                                                 \
         return atomic_compare_exchange_weak_explicit((_z_atomic_##name##_atomic_t *)&var->_value, expected, desired,   \
                                                      _z_atomic_memory_order(success),                                  \
                                                      _z_atomic_memory_order(failure));                                 \

zephyr/CMakeLists.txt

@@ -3,6 +3,9 @@ if(CONFIG_ZENOH_PICO)
 zephyr_compile_definitions(ZENOH_ZEPHYR)
 zephyr_include_directories(../include)
 zephyr_library()
+if(CONFIG_COMPILER_GCC)
+  zephyr_link_libraries(atomic)
+endif()
 
 function(configure_zenoh_feature config)
   string(REPLACE CONFIG_ZENOH_PICO Z_FEATURE feature ${config})