From fe08046d41d80bb2efff148a5e0157a1e8c95006 Mon Sep 17 00:00:00 2001 From: Jay Conrod Date: Tue, 21 Sep 2021 13:54:58 -0700 Subject: [PATCH] cmd/go: adjust comments on why fuzzing instrumentation is disabled For #48504 Related #14565 Change-Id: Ibe43c75224525c4b80dbb66a1b6e0d688e47e2e4 Reviewed-on: https://go-review.googlesource.com/c/go/+/351314 Trust: Jay Conrod Trust: Katie Hockman Run-TryBot: Jay Conrod TryBot-Result: Go Bot Reviewed-by: Katie Hockman --- fuzz/counters_unsupported.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/fuzz/counters_unsupported.go b/fuzz/counters_unsupported.go index 743ef45..9595cb9 100644 --- a/fuzz/counters_unsupported.go +++ b/fuzz/counters_unsupported.go @@ -7,7 +7,9 @@ package fuzz // TODO(#48504): re-enable on platforms where instrumentation works. -// This was disabled due to an init failure on aix_ppc64. +// In theory, we shouldn't need this file at all: if the binary was built +// without coverage, then _counters and _ecounters should have the same address. +// However, this caused an init failure on aix/ppc64, so it's disabled here. // coverage returns a []byte containing unique 8-bit counters for each edge of // the instrumented source code. This coverage data will only be generated if