]> Cypherpunks repositories - gostls13.git/commit
test: move load/store combines into asmcheck
authorGiovanni Bajo <rasky@develer.com>
Sat, 3 Mar 2018 15:00:43 +0000 (16:00 +0100)
committerGiovanni Bajo <rasky@develer.com>
Sun, 4 Mar 2018 16:52:03 +0000 (16:52 +0000)
commitfad31e513d721a36f2d881d99de54def805a8648
treebfc309f2c9bba94728a42a69e0f35770d5ccb61f
parent80bfb75c429c4930b8ed603592b4c9df82a79240
test: move load/store combines into asmcheck

This CL moves the load/store combining tests into asmcheck.
In addition at being more compact, it's also now easier to
spot what it is missing in each architecture.

While doing so, I think I uncovered a bug in ppc64le and arm64
rules, because they fail to load/store combine in non-trivial
functions. Not sure why, I'll open an issue.

Change-Id: Ia1572d53c0553d9104f3e52b95e4d1768a8440a3
Reviewed-on: https://go-review.googlesource.com/98441
Run-TryBot: Giovanni Bajo <rasky@develer.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
src/cmd/compile/internal/gc/asm_test.go
test/codegen/memcombine.go [new file with mode: 0644]