]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: do not use "oaslit" for global
authorCherry Zhang <cherryyz@google.com>
Mon, 6 Feb 2017 19:24:16 +0000 (14:24 -0500)
committerCherry Zhang <cherryyz@google.com>
Tue, 7 Feb 2017 17:23:23 +0000 (17:23 +0000)
commit160914e33ca6521d74297291d801062cc44d794d
treeb6baf800c5640c456cd4a099ba8207d28f4aa688
parent1ead0bd1dc8958939b16b8fc3ab2cc8242f5e831
cmd/compile: do not use "oaslit" for global

The compiler did not emit write barrier for assigning global with
struct literal, like global = T{} where T contains pointer.

The relevant code path is:
walkexpr OAS var_ OSTRUCTLIT
    oaslit
        anylit OSTRUCTLIT
            walkexpr OAS var_ nil
            return without adding write barrier
    return true
break (without adding write barrier)

This CL makes oaslit not apply to globals. See also CL
https://go-review.googlesource.com/c/36355/ for an alternative
fix.

The downside of this is that it generates static data for zeroing
struct now. Also this only covers global. If there is any lurking
bug with implicit zeroing other than globals, this doesn't fix.

Fixes #18956.

Change-Id: Ibcd27e4fae3aa38390ffa94a32a9dd7a802e4b37
Reviewed-on: https://go-review.googlesource.com/36410
Reviewed-by: Russ Cox <rsc@golang.org>
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
src/cmd/compile/internal/gc/sinit.go
test/writebarrier.go