From 130ede0d9e01ef53e734371faea080f5301d9c55 Mon Sep 17 00:00:00 2001 From: Than McIntosh Date: Mon, 6 Jul 2020 09:38:17 -0400 Subject: [PATCH] [dev.link] cmd/link: remove some unneeded code from writeBlock() The loader writeBlock() function has code that tries to skip the initial portion of the input symbols list depending on the address of the section being written-- this code is dead (skipping is never triggered) due to similar skipping in the callers; remove this preamble. Change-Id: I9769694a3194faf73ebebbbc10ceba4928c3087c Reviewed-on: https://go-review.googlesource.com/c/go/+/241067 Run-TryBot: Than McIntosh TryBot-Result: Gobot Gobot Reviewed-by: Jeremy Faller --- src/cmd/link/internal/ld/data.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/cmd/link/internal/ld/data.go b/src/cmd/link/internal/ld/data.go index e6ca977991..d9f9ff70d3 100644 --- a/src/cmd/link/internal/ld/data.go +++ b/src/cmd/link/internal/ld/data.go @@ -878,12 +878,6 @@ func writeBlocks(ctxt *Link, out *OutBuf, sem chan int, ldr *loader.Loader, syms } func writeBlock(ctxt *Link, out *OutBuf, ldr *loader.Loader, syms []loader.Sym, addr, size int64, pad []byte) { - for i, s := range syms { - if ldr.SymValue(s) >= addr && !ldr.AttrSubSymbol(s) { - syms = syms[i:] - break - } - } st := ctxt.makeRelocSymState() -- 2.50.0