diff options
| author | Keith Randall <khr@golang.org> | 2017-02-13 09:37:06 -0800 |
|---|---|---|
| committer | Keith Randall <khr@golang.org> | 2017-02-13 18:29:51 +0000 |
| commit | b548eee3d96fc0b6e962a243b28121e1f37ad792 (patch) | |
| tree | 8e960a95d546c5d069969ef6494916dcd2f51ac5 /src/database/sql/sql_test.go | |
| parent | 76b4b8c72dc319454ff3ecb83bf49831e4e528c3 (diff) | |
| download | go-b548eee3d96fc0b6e962a243b28121e1f37ad792.tar.xz | |
cmd/compile: fix load-combining rules
CL 33632 reorders args of commutative ops in order to make
CSE for commutative ops more robust. Unfortunately, that
broke the load-combining rules which depend on a certain ordering
of OR ops' arguments.
Introduce some additional rules that order OR ops' arguments
consistently so that the load-combining rules fire.
Note: there's also something else wrong with the s390x rules.
I've filed #19059 for that.
Fixes #18946
Change-Id: I0a5447196bd88a55ccee683c69a57b943a9972e1
Reviewed-on: https://go-review.googlesource.com/36911
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Josh Bleecher Snyder <josharian@gmail.com>
Diffstat (limited to 'src/database/sql/sql_test.go')
0 files changed, 0 insertions, 0 deletions