From ca5cea9dca0bc635b9a23cfe65226f3e1a423342 Mon Sep 17 00:00:00 2001 From: Josh Bleecher Snyder Date: Wed, 2 Nov 2016 17:55:53 -0700 Subject: [PATCH] cmd/compile: add OMOD to list of ops that might panic Follow-up to CL 32551. Change-Id: If68f9581a7f13e04796aaff2007c09f8ea2c3611 Reviewed-on: https://go-review.googlesource.com/32586 Run-TryBot: Josh Bleecher Snyder TryBot-Result: Gobot Gobot Reviewed-by: Keith Randall --- src/cmd/compile/internal/gc/subr.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cmd/compile/internal/gc/subr.go b/src/cmd/compile/internal/gc/subr.go index 7655a9ecdc..fb5419f533 100644 --- a/src/cmd/compile/internal/gc/subr.go +++ b/src/cmd/compile/internal/gc/subr.go @@ -1183,7 +1183,7 @@ func ullmancalc(n *Node) { goto out } case OINDEX, OSLICE, OSLICEARR, OSLICE3, OSLICE3ARR, OSLICESTR, - OIND, ODOTPTR, ODOTTYPE, ODIV: + OIND, ODOTPTR, ODOTTYPE, ODIV, OMOD: // These ops might panic, make sure they are done // before we start marshaling args for a call. See issue 16760. ul = UINF -- 2.48.1