build: link V8 with atomic library

Closes: #290
nodejs · Oct 21, 2024 · 70d6ae4 · 70d6ae4
1 parent 27a8311
commit 70d6ae4
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/tools/v8_gypfiles/v8.gyp b/tools/v8_gypfiles/v8.gyp
@@ -1222,8 +1222,9 @@
           'dependencies': ['postmortem-metadata#target'],
         }],
         # Platforms that don't have Compare-And-Swap (CAS) support need to link atomic library
-        # to implement atomic memory access
-        ['v8_current_cpu in ["mips64", "mips64el", "arm", "riscv64", "loong64"]', {
+        # to implement atomic memory access.
+        # Clang needs it for some atomic operations (https://clang.llvm.org/docs/Toolchain.html#atomics-library).
+        ['(OS=="linux" and clang==1) or (v8_current_cpu in ["mips64", "mips64el", "arm", "riscv64", "loong64"])', {
           'link_settings': {
             'libraries': ['-latomic', ],
           },