From: Daniel Martí <mvdan@mvdan.cc>
Date: Thu, 28 Sep 2017 19:44:21 +0000 (+0100)
Subject: cmd/compile: add reflect to TestIntendedInlining
X-Git-Tag: go1.10beta1~940
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=a2993456bb765e9a4c556e3529a01975764d28ae;p=gostls13.git

cmd/compile: add reflect to TestIntendedInlining

Add the package to the table and start it off with a few small, basic
functions. Inspired by CL 66331, which added flag.ro.

Updates #21851.

Change-Id: I3995cde1ff7bb09a718110473bed8b193c2232a5
Reviewed-on: https://go-review.googlesource.com/66990
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ilya Tocar <ilya.tocar@intel.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
---

diff --git a/src/cmd/compile/internal/gc/inl_test.go b/src/cmd/compile/internal/gc/inl_test.go
index 426417ce79..3fdbf1e492 100644
--- a/src/cmd/compile/internal/gc/inl_test.go
+++ b/src/cmd/compile/internal/gc/inl_test.go
@@ -113,6 +113,23 @@ func TestIntendedInlining(t *testing.T) {
 			"RuneLen",
 			"ValidRune",
 		},
+		"reflect": {
+			"Value.CanAddr",
+			"Value.CanSet",
+			"Value.IsValid",
+			"add",
+			"align",
+			"flag.kind",
+			"flag.ro",
+
+			// TODO: these use panic, need mid-stack
+			// inlining
+			// "Value.CanInterface",
+			// "Value.pointer",
+			// "flag.mustBe",
+			// "flag.mustBeAssignable",
+			// "flag.mustBeExported",
+		},
 		"regexp": {
 			"(*bitState).push",
 		},