Skip to content

Commit

Permalink
build: remove unused libatomic on ppc64, s390x
Browse files Browse the repository at this point in the history
The library is not necessary for ppc64 or s390x. It does no harm with
some linkers, but devtoolset-6 creates runtime dependencies on all link
libraries, even unused ones.

Fixes: nodejs#27377
Fixes: nodejs#29718
  • Loading branch information
sam-github committed Oct 1, 2019
1 parent a8956e4 commit e2d50ab
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tools/v8_gypfiles/v8.gyp
Expand Up @@ -795,7 +795,7 @@
}],
# Platforms that don't have Compare-And-Swap (CAS) support need to link atomic library
# to implement atomic memory access
['v8_current_cpu in ["mips", "mipsel", "mips64", "mips64el", "ppc", "ppc64", "s390x"]', {
['v8_current_cpu in ["mips", "mipsel", "mips64", "mips64el", "ppc"]', {
'link_settings': {
'libraries': ['-latomic', ],
},
Expand Down

0 comments on commit e2d50ab

Please sign in to comment.