cmd/link: don't pass -Wl,-S on AIX

cherrymui · cherrymui · commit 459f3a3adc13 · 2025-09-26T10:15:40.000-07:00
The AIX linker's -S flag has a different meaning. Don't pass it. Updates golang#75618. Change-Id: I98faabea3435cde255f4c2d25f34dde9f69b7ec9 Reviewed-on: https://go-review.googlesource.com/c/go/+/707097 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: David Chase <drchase@google.com>
diff --git a/src/cmd/link/internal/ld/lib.go b/src/cmd/link/internal/ld/lib.go
@@ -1452,7 +1452,9 @@ func (ctxt *Link) hostlink() {
 			argv = append(argv, "-s")
 		}
 	} else if *FlagW {
-		argv = append(argv, "-Wl,-S") // suppress debugging symbols
+		if !ctxt.IsAIX() { // The AIX linker's -S has different meaning
+			argv = append(argv, "-Wl,-S") // suppress debugging symbols
+		}
 	}
 
 	// On darwin, whether to combine DWARF into executable.

Original file line number	Diff line number	Diff line change
`@@ -1452,7 +1452,9 @@ func (ctxt *Link) hostlink() {`
`1452`	`1452`	`argv = append(argv, "-s")`
`1453`	`1453`	`}`
`1454`	`1454`	`} else if *FlagW {`
`1455`		`- argv = append(argv, "-Wl,-S") // suppress debugging symbols`
	`1455`	`+ if !ctxt.IsAIX() { // The AIX linker's -S has different meaning`
	`1456`	`+ argv = append(argv, "-Wl,-S") // suppress debugging symbols`
	`1457`	`+ }`
`1456`	`1458`	`}`
`1457`	`1459`
`1458`	`1460`	`// On darwin, whether to combine DWARF into executable.`