build: link V8 with atomic library

Closes: nodejs/node-v8#290
nodejs · Sep 19, 2024 · d48a069 · d48a069
1 parent 8913cec
commit d48a069
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', ],
           },