Commit e2c9cab
committed
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 <targos@protonmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>1 parent c5d68c4 commit e2c9cab
1 file changed
Lines changed: 12 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -542,7 +542,12 @@ | |||
| 542 | 542 | 'ldflags': [ '-m32' ], | |
| 543 | 543 | }], | |
| 544 | 544 | [ 'host_arch=="ppc64" and OS not in "aix os400"', { | |
| 545 | - 'cflags': [ '-m64', '-mminimal-toc' ], | ||
| 545 | + 'conditions': [ | ||
| 546 | + [ 'clang==0', { | ||
| 547 | + 'cflags': [ '-mminimal-toc' ], | ||
| 548 | + }], | ||
| 549 | + ], | ||
| 550 | + 'cflags': [ '-m64' ], | ||
| 546 | 551 | 'ldflags': [ '-m64' ], | |
| 547 | 552 | }], | |
| 548 | 553 | [ 'host_arch=="s390x" and OS=="linux"', { | |
@@ -566,7 +571,12 @@ | |||
| 566 | 571 | 'ldflags': [ '-m32' ], | |
| 567 | 572 | }], | |
| 568 | 573 | [ 'target_arch=="ppc64" and OS not in "aix os400"', { | |
| 569 | - 'cflags': [ '-m64', '-mminimal-toc' ], | ||
| 574 | + 'conditions': [ | ||
| 575 | + [ 'clang==0', { | ||
| 576 | + 'cflags': [ '-mminimal-toc' ], | ||
| 577 | + }], | ||
| 578 | + ], | ||
| 579 | + 'cflags': [ '-m64' ], | ||
| 570 | 580 | 'ldflags': [ '-m64' ], | |
| 571 | 581 | }], | |
| 572 | 582 | [ 'target_arch=="s390x" and OS=="linux"', { | |
0 commit comments