From 0ef74f37a55d7c0f468a69069f17a13e046b002f Mon Sep 17 00:00:00 2001 From: Josh Bleecher Snyder Date: Tue, 15 Mar 2022 16:00:27 -0700 Subject: [PATCH] net/dns/resolver: remove closure allocation explanation As of Go 1.18, the register ABI list includes arm64, amd64, ppc64, and ppc64le. This is a large enough percentage of the architectures that it's not worth explaining. Signed-off-by: Josh Bleecher Snyder --- net/dns/resolver/tsdns_test.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/net/dns/resolver/tsdns_test.go b/net/dns/resolver/tsdns_test.go index ec5fb37af..2bd21007c 100644 --- a/net/dns/resolver/tsdns_test.go +++ b/net/dns/resolver/tsdns_test.go @@ -965,8 +965,6 @@ func TestAllocs(t *testing.T) { }{ // Name lowercasing, response slice created by dns.NewBuilder, // and closure allocation from go call. - // (Closure allocation only happens when using new register ABI, - // which is amd64 with Go 1.17, and probably more platforms later.) {"forward", dnspacket("test1.ipn.dev.", dns.TypeA, noEdns), 3}, // 3 extra allocs in rdnsNameToIPv4 and one in marshalPTRRecord (dns.NewName). {"reverse", dnspacket("4.3.2.1.in-addr.arpa.", dns.TypePTR, noEdns), 5},