]> Cypherpunks repositories - gostls13.git/commit
spec: clarify restrictions on RHS of non-constant shifts
authorRobert Griesemer <gri@golang.org>
Wed, 7 Jun 2017 18:06:00 +0000 (11:06 -0700)
committerRobert Griesemer <gri@golang.org>
Thu, 8 Jun 2017 16:46:15 +0000 (16:46 +0000)
commite4ce08afe0a01b850e266efef52b03eab94a470f
tree9dbdef3fa84c5f6bc878cd9b86c0dc48d9a4767e
parenta7d7d7aec26b18e8ae4581a30b7c0f5b4d0b9c6b
spec: clarify restrictions on RHS of non-constant shifts

For non-constant shifts with an untyped constant shift count, the
spec only said that it must "be converted to unsigned integer type".
go/types accepts any (arbitrarily large) integer value. Both cmd/compile
and gccgo require that the shift count be representable as a uint value
in that case (if the shift count is typed, it may be any unsigned integer
type).

This change adjusts the spec to state what the compilers have been doing
all along. The new wording matches similar rules elsewhere (e.g., for
untyped array and slice indices). Also, while technically this is a
restriction (we could permit arbitrarily large shift counts), in practice
this is irrelevant.

Fixes #14822.

Change-Id: Ia75834c67483cf761c10025c8df758f225ef67c2
Reviewed-on: https://go-review.googlesource.com/45072
Reviewed-by: Ian Lance Taylor <iant@golang.org>
doc/go_spec.html