tree 6f7f7c31f8c62ef31a33d19d45f8243b697eabfa
parent eeb319a5a5c9f1ef7e31c551b03ec841ee9f0b61
author Cherry Zhang <cherryyz@google.com> 1575590214 -0500
committer Cherry Zhang <cherryyz@google.com> 1575593831 +0000

cmd/compile: don't fuse branches with side effects

Count Values with side effects but no use as live, and don't fuse
branches that contain such Values. (This can happen e.g. when it
is followed by an infinite loop.) Otherwise this may lead to
miscompilation (side effect fired at wrong condition) or ICE (two
stores live simultaneously).

Fixes #36005.

Change-Id: If202eae4b37cb7f0311d6ca120ffa46609925157
Reviewed-on: https://go-review.googlesource.com/c/go/+/210179
Reviewed-by: Keith Randall <khr@golang.org>