diff options
author | Todd Neal <todd@tneal.org> | 2017-04-24 11:20:09 -0400 |
---|---|---|
committer | Brad Fitzpatrick <bradfitz@golang.org> | 2017-05-23 21:25:37 +0000 |
commit | 195e20a9768468d580231e62e37e6fb0525ed20e (patch) | |
tree | 1884a5cfc81fa00791bc407acf457c8fd6d801a1 | |
parent | f55bc1c4ebea3feffed484b7fa8cddb4cd07c1c3 (diff) | |
download | go-195e20a9768468d580231e62e37e6fb0525ed20e.tar.gz go-195e20a9768468d580231e62e37e6fb0525ed20e.zip |
[release-branch.go1.8] cmd/compile: ignore types when considering tuple select for CSE
Fixes #20097
Change-Id: I3c9626ccc8cd0c46a7081ea8650b2ff07a5d4fcd
Reviewed-on: https://go-review.googlesource.com/41505
Run-TryBot: Todd Neal <todd@tneal.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-on: https://go-review.googlesource.com/43997
Run-TryBot: Chris Broadfoot <cbro@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-rw-r--r-- | src/cmd/compile/internal/ssa/cse.go | 10 | ||||
-rw-r--r-- | test/fixedbugs/issue20097.go | 18 |
2 files changed, 25 insertions, 3 deletions
diff --git a/src/cmd/compile/internal/ssa/cse.go b/src/cmd/compile/internal/ssa/cse.go index 4e07c89b88..9ab18d89e9 100644 --- a/src/cmd/compile/internal/ssa/cse.go +++ b/src/cmd/compile/internal/ssa/cse.go @@ -313,9 +313,13 @@ func cmpVal(v, w *Value, auxIDs auxmap) Cmp { // that generate memory. return lt2Cmp(v.ID < w.ID) } - - if tc := v.Type.Compare(w.Type); tc != CMPeq { - return tc + // OpSelect is a pseudo-op. We need to be more agressive + // regarding CSE to keep multiple OpSelect's of the same + // argument from existing. + if v.Op != OpSelect0 && v.Op != OpSelect1 { + if tc := v.Type.Compare(w.Type); tc != CMPeq { + return tc + } } if v.Aux != w.Aux { diff --git a/test/fixedbugs/issue20097.go b/test/fixedbugs/issue20097.go new file mode 100644 index 0000000000..236a1bd40a --- /dev/null +++ b/test/fixedbugs/issue20097.go @@ -0,0 +1,18 @@ +// compile + +// Copyright 2017 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +// Issue 20097: ensure that we CSE multiple Select ops with +// the same underlying type + +package main + +type T int64 + +func f(x, y int64) (int64, T) { + a := x / y + b := T(x) / T(y) + return a, b +} |