]> Cypherpunks.ru repositories - gostls13.git/commit
cmd/compile/internal/ssa: re-adjust CarryChainTail scheduling priority
authorPaul E. Murphy <murp@ibm.com>
Fri, 28 Oct 2022 20:59:43 +0000 (15:59 -0500)
committerPaul Murphy <murp@ibm.com>
Thu, 3 Nov 2022 19:59:19 +0000 (19:59 +0000)
commitd031e9e07a07afef8d16576fd7079a739a7e4394
tree1adb413f7109ac22b7b25a05fe743e43f56c25d1
parentc065bc70ef45003216271ba42bdd4a4c321ac38a
cmd/compile/internal/ssa: re-adjust CarryChainTail scheduling priority

This needs to be as low as possible while not breaking priority
assumptions of other scores to correctly schedule carry chains.

Prior to the arm64 changes, it was set below ReadTuple. At the time,
this prevented the MulHiLo implementation on PPC64 from occluding
the scheduling of a full carry chain.

Memory scores can also prevent better scheduling, as can be observed
with crypto/internal/edwards25519/field.feMulGeneric.

Fixes #56497

Change-Id: Ia4b54e6dffcce584faf46b1b8d7cea18a3913887
Reviewed-on: https://go-review.googlesource.com/c/go/+/447435
Reviewed-by: Cherry Mui <cherryyz@google.com>
Reviewed-by: Keith Randall <khr@google.com>
Run-TryBot: Paul Murphy <murp@ibm.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Bryan Mills <bcmills@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
src/cmd/compile/internal/ssa/schedule.go
test/codegen/mathbits.go