Skip to content

Commit e5b9911

Browse files
targosnodejs-github-bot
authored andcommitted
build: link V8 with atomic library
Closes: #290
1 parent a2d83c0 commit e5b9911

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

tools/v8_gypfiles/v8.gyp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1249,8 +1249,9 @@
12491249
'dependencies': ['postmortem-metadata#target'],
12501250
}],
12511251
# Platforms that don't have Compare-And-Swap (CAS) support need to link atomic library
1252-
# to implement atomic memory access
1253-
['v8_current_cpu in ["mips64", "mips64el", "arm", "riscv64", "loong64"]', {
1252+
# to implement atomic memory access.
1253+
# Clang needs it for some atomic operations (https://clang.llvm.org/docs/Toolchain.html#atomics-library).
1254+
['(OS=="linux" and clang==1) or (v8_current_cpu in ["mips64", "mips64el", "arm", "riscv64", "loong64"])', {
12541255
'link_settings': {
12551256
'libraries': ['-latomic', ],
12561257
},

0 commit comments

Comments
 (0)