Skip to content

Commit 105f878

Browse files
committed
libmetal/atomic: enable 64-bit atomic by toolchain builtin flags
Fix compile error: arm-none-eabi-ld: (remoteproc_virtio.o): in function `metal_io_read': metal/io.h:252: undefined reference to `__atomic_load_8' arm-none-eabi-ld: (remoteproc_virtio.o): in function `metal_io_write': metal/io.h:290: undefined reference to `__atomic_store_8' Not all 32-bit architectures support 64bit atomic, gcc/clang toolchains have built-in properties to indicate whether support atomic64: | $ arm-none-eabi-gcc -march=armv7e-m -dM -E - < /dev/null | grep SYNC | #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_1 1 | #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_2 1 | #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_4 1 Signed-off-by: chao an <[email protected]>
1 parent 3aee6be commit 105f878

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

lib/io.h

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,9 @@ extern "C" {
3030
* @{
3131
*/
3232

33-
#ifdef __MICROBLAZE__
34-
#define NO_ATOMIC_64_SUPPORT
33+
#if defined(__MICROBLAZE__) || \
34+
(defined(__GNUC__) && !defined(__GCC_HAVE_SYNC_COMPARE_AND_SWAP_8))
35+
# define NO_ATOMIC_64_SUPPORT
3536
#endif
3637

3738
struct metal_io_region;

0 commit comments

Comments
 (0)