commit 63c6481f3be3d4c29183574fa76516c4e7f54c6e
author Alan Donovan <adonovan@google.com> Mon Jun 19 13:33:23 2017 -0400
committer Alan Donovan <adonovan@google.com> Mon Jun 19 17:45:44 2017 +0000
tree 6df75be1b72fa20285c1979973954124f71fd7c3
parent 5128de728888ea5cc98be50b4efaabc4d3491274

go/loader: fix a data race

The loader was calling (*types.Checker).Files on the "unsafe" package,
a global variable.  Even with zero files, this operation is not a no-op
because it sets the package's "complete" flag, leading to a data race.
(Because Unsafe.complete is already set at construction, the
race is benign, but is reported by -race nonetheless.)

Fixes golang/go#20718

Change-Id: I5a4f95be5ab4c60ea3b6c2a7fb6f1b67acbf42bc
Reviewed-on: https://go-review.googlesource.com/46071
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>