Skip to content

Commit 89dd770

Browse files
richardlautargos
authored andcommitted
build: do not set -mminimal-toc with clang
This is a gcc-only option, do not pass to clang. PR-URL: #59484 Refs: nodejs/build#4091 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
1 parent b4a23d6 commit 89dd770

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

common.gypi

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -538,7 +538,12 @@
538538
'ldflags': [ '-m64' ],
539539
}],
540540
[ 'host_arch=="ppc64" and OS not in "aix os400"', {
541-
'cflags': [ '-m64', '-mminimal-toc' ],
541+
'conditions': [
542+
[ 'clang==0', {
543+
'cflags': [ '-mminimal-toc' ],
544+
}],
545+
],
546+
'cflags': [ '-m64' ],
542547
'ldflags': [ '-m64' ],
543548
}],
544549
[ 'host_arch=="s390x" and OS=="linux"', {
@@ -558,7 +563,12 @@
558563
'ldflags': [ '-m64' ],
559564
}],
560565
[ 'target_arch=="ppc64" and OS not in "aix os400"', {
561-
'cflags': [ '-m64', '-mminimal-toc' ],
566+
'conditions': [
567+
[ 'clang==0', {
568+
'cflags': [ '-mminimal-toc' ],
569+
}],
570+
],
571+
'cflags': [ '-m64' ],
562572
'ldflags': [ '-m64' ],
563573
}],
564574
[ 'target_arch=="s390x" and OS=="linux"', {

0 commit comments

Comments
 (0)