[dev.link] cmd/link: fix buglet in setupdynexp

This should restore deterministic order of dynexp, and fix
Solaris build.

Change-Id: Icb796babaa3238bff90fd8255ee9f023f2306c26
Reviewed-on: https://go-review.googlesource.com/c/go/+/225538
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Than McIntosh <thanm@google.com>

diff --git a/src/cmd/link/internal/ld/lib.go b/src/cmd/link/internal/ld/lib.go
index 01dc31b4d8cce7a06f842547a102f2739b7c89ba..88c51d65b3f68cbd7aa8705e5a7fa11ff550034b 100644 (file)
--- a/src/cmd/link/internal/ld/lib.go
+++ b/src/cmd/link/internal/ld/lib.go
@@ -620,7 +620,7 @@ func setupdynexp(ctxt *Link) {
                        panic("dynexp entry not reachable")
                }
        }
-       sort.Slice(dynexp, func(i, j int) bool {
+       sort.Slice(d, func(i, j int) bool {
                return ctxt.loader.SymName(d[i]) < ctxt.loader.SymName(d[j])
        })