cmd/compile/internal/test: skip testpoint due to revert of CL 479095

Skip one of the testpoints that verifies inlining, since it
no longer passes as a result of reverting CL 479095. Once we
roll forward with a new version of CL 479095 we can re-enable
this testpoint.

Change-Id: I41f6fb3fce78f31e60c5f0ed2856be0e66865149
Reviewed-on: https://go-review.googlesource.com/c/go/+/481755
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Than McIntosh <thanm@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>

diff --git a/src/cmd/compile/internal/test/inl_test.go b/src/cmd/compile/internal/test/inl_test.go
index 205b746dd8925f31c4ca0a9f56e597949793bcc9..2a16b21cef76ea77dacd493da4ed6da86ab0cca9 100644 (file)
--- a/src/cmd/compile/internal/test/inl_test.go
+++ b/src/cmd/compile/internal/test/inl_test.go
@@ -180,15 +180,19 @@ func TestIntendedInlining(t *testing.T) {
                "net": {
                        "(*UDPConn).ReadFromUDP",
                },
-               "sync": {
-                       // Both OnceFunc and its returned closure need to be inlinable so
-                       // that the returned closure can be inlined into the caller of OnceFunc.
-                       "OnceFunc",
-                       "OnceFunc.func2", // The returned closure.
-                       // TODO(austin): It would be good to check OnceValue and OnceValues,
-                       // too, but currently they aren't reported because they have type
-                       // parameters and aren't instantiated in sync.
-               },
+               // These testpoints commented out for now, since CL 479095
+               // had to be reverted. We can re-enable this once we roll
+               // forward with a new version of 479095.
+               /*
+                       "sync": {
+                               // Both OnceFunc and its returned closure need to be inlinable so
+                               // that the returned closure can be inlined into the caller of OnceFunc.
+                               "OnceFunc",
+                               "OnceFunc.func2", // The returned closure.
+                               // TODO(austin): It would be good to check OnceValue and OnceValues,
+                               // too, but currently they aren't reported because they have type
+                               // parameters and aren't instantiated in sync.
+                       }, */
                "sync/atomic": {
                        // (*Bool).CompareAndSwap handled below.
                        "(*Bool).Load",