From 433be563b6246eb132aed6e9e58f46a0d46f7010 Mon Sep 17 00:00:00 2001 From: Andrew Pogrebnoy Date: Tue, 4 Oct 2016 02:39:33 +0300 Subject: [PATCH] cmd/compile: fix choice of phi building algorithm The algorithm for placing a phi nodes in small functions now unreachable. This patch fix that. Change-Id: I253d745b414fa12ee0719459c28e78a69c6861ae Reviewed-on: https://go-review.googlesource.com/30106 Run-TryBot: Brad Fitzpatrick TryBot-Result: Gobot Gobot Reviewed-by: Keith Randall --- src/cmd/compile/internal/gc/phi.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cmd/compile/internal/gc/phi.go b/src/cmd/compile/internal/gc/phi.go index ea9e5b10aa..0d4dbb5b2e 100644 --- a/src/cmd/compile/internal/gc/phi.go +++ b/src/cmd/compile/internal/gc/phi.go @@ -28,7 +28,7 @@ const debugPhi = false // of the appropriate phi or definition. // TODO: make this part of cmd/compile/internal/ssa somehow? func (s *state) insertPhis() { - if len(s.f.Blocks) <= smallBlocks && false { + if len(s.f.Blocks) <= smallBlocks { sps := simplePhiState{s: s, f: s.f, defvars: s.defvars} sps.insertPhis() return