diff options
| author | Cuong Manh Le <cuong.manhle.vn@gmail.com> | 2025-08-04 18:48:43 +0700 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2025-08-05 08:32:51 -0700 |
| commit | 26da1199ebdddeebc5eebdd562d8a40f01f42a1b (patch) | |
| tree | 7fc3b5f9fbfd450ba6ba37f9c03e6616e72a447b /src/cmd/internal | |
| parent | 5ab9f23977694ab87614fd30e081d294eb313efd (diff) | |
| download | go-26da1199ebdddeebc5eebdd562d8a40f01f42a1b.tar.xz | |
cmd/compile: make isUint{32,64}PowerOfTwo implementations clearer
Since these functions cast the input to uint64, so the result always
non-negative. The condition should be changed to comparing with zero,
thus maaking it clearer to reader, and open room for simplifying in the
future by using the generic isUnsignedPowerOfTwo function.
Separated this change, so it's easier to do bisecting if there's any
problems happened.
Change-Id: Ibec28c2590f4c52caa36384b710d526459725e49
Reviewed-on: https://go-review.googlesource.com/c/go/+/692915
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Reviewed-by: Keith Randall <khr@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Keith Randall <khr@google.com>
Diffstat (limited to 'src/cmd/internal')
0 files changed, 0 insertions, 0 deletions