]> Cypherpunks repositories - gostls13.git/commitdiff
doc: add Go 1.17 release note about inlining functions with closures
authorDan Scales <danscales@google.com>
Sat, 22 May 2021 20:40:20 +0000 (13:40 -0700)
committerDan Scales <danscales@google.com>
Tue, 25 May 2021 19:03:32 +0000 (19:03 +0000)
Fixes #45781

Change-Id: Ia5bc2845f7f94aff4f3f0ff15533feb148223adb
Reviewed-on: https://go-review.googlesource.com/c/go/+/322089
Trust: Dan Scales <danscales@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
doc/go1.17.html

index 6dd1d0d1db5575d61ff93bb75f2260f21eee1d91..c2317a4035284d21e8194c903a7c0f565ec3b55d 100644 (file)
@@ -228,7 +228,14 @@ Do not send CLs removing the interior tags from such phrases.
 
 <h2 id="compiler">Compiler</h2>
 
-<p>
+<p><!-- CL 283112, golang.org/issue/28727 -->
+
+  Functions containing closures can now be inlined. One effect of this change is
+  that a function with a closure may actually produce a distinct closure function
+  for each place that the function is inlined. Hence, this change could reveal
+  bugs where Go functions are compared (incorrectly) by pointer value. Go
+  functions are by definition not comparable.
+
   TODO: complete the Compiler section, or delete if not needed
 </p>