aboutsummaryrefslogtreecommitdiff
path: root/test/writebarrier.go
diff options
context:
space:
mode:
authorJosh Bleecher Snyder <josharian@gmail.com>2017-05-10 12:48:17 -0700
committerJosh Bleecher Snyder <josharian@gmail.com>2017-05-11 18:28:40 +0000
commitee69c21747b40f79351dab63b5ac6715c86e04f2 (patch)
tree480b7baa03dc06d37e4d52e34a555ca073d158ad /test/writebarrier.go
parentdccc653aeb35d511691191517c021c2ce6125cb4 (diff)
downloadgo-ee69c21747b40f79351dab63b5ac6715c86e04f2.tar.gz
go-ee69c21747b40f79351dab63b5ac6715c86e04f2.zip
cmd/compile: don't use statictmps for SSA-able composite literals
The writebarrier test has to change. Now that T23 composite literals are passed to the backend, they get SSA'd, so writes to their fields are treated separately, so the relevant part of the first write to t23 is now a dead store. Preserve the intent of the test by splitting it up into two functions. Reduces code size a bit: name old object-bytes new object-bytes delta Template 386k ± 0% 386k ± 0% ~ (all equal) Unicode 202k ± 0% 202k ± 0% ~ (all equal) GoTypes 1.16M ± 0% 1.16M ± 0% ~ (all equal) Compiler 3.92M ± 0% 3.91M ± 0% -0.19% (p=0.008 n=5+5) SSA 7.91M ± 0% 7.91M ± 0% ~ (all equal) Flate 228k ± 0% 228k ± 0% -0.05% (p=0.008 n=5+5) GoParser 283k ± 0% 283k ± 0% ~ (all equal) Reflect 952k ± 0% 952k ± 0% -0.06% (p=0.008 n=5+5) Tar 188k ± 0% 188k ± 0% -0.09% (p=0.008 n=5+5) XML 406k ± 0% 406k ± 0% -0.02% (p=0.008 n=5+5) [Geo mean] 649k 648k -0.04% Fixes #18872 Change-Id: Ifeed0f71f13849732999aa731cc2bf40c0f0e32a Reviewed-on: https://go-review.googlesource.com/43154 Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: David Chase <drchase@google.com> Reviewed-by: Cherry Zhang <cherryyz@google.com>
Diffstat (limited to 'test/writebarrier.go')
-rw-r--r--test/writebarrier.go5
1 files changed, 4 insertions, 1 deletions
diff --git a/test/writebarrier.go b/test/writebarrier.go
index f3149e1b49..55ba81e764 100644
--- a/test/writebarrier.go
+++ b/test/writebarrier.go
@@ -238,11 +238,14 @@ var i23 int
// f23x: zeroing global needs write barrier for the hybrid barrier.
func f23a() {
t23 = T23{} // ERROR "write barrier"
+}
+
+func f23b() {
// also test partial assignments
t23 = T23{a: 1} // ERROR "write barrier"
}
-func f23b() {
+func f23c() {
t23 = T23{} // no barrier (dead store)
// also test partial assignments
t23 = T23{p: &i23} // ERROR "write barrier"