diff options
author | Bill O'Farrell <billo@ca.ibm.com> | 2017-10-27 14:40:59 -0400 |
---|---|---|
committer | Michael Munday <mike.munday@ibm.com> | 2017-10-30 23:42:27 +0000 |
commit | 7fff1db0605739fee20673475cbc1813fdf7008e (patch) | |
tree | 8f0acb7302d9c77ba4e54a36b31bd13f4063d47b /src/runtime/asm_s390x.s | |
parent | 03c8c566829d4bdc7a1c2053124643e32a6d2365 (diff) | |
download | go-7fff1db0605739fee20673475cbc1813fdf7008e.tar.gz go-7fff1db0605739fee20673475cbc1813fdf7008e.zip |
runtime: remove unnecessary sync from publicationBarrier on s390x
Memory accesses on z are at least as ordered as they are on AMD64.
Change-Id: Ia515430e571ebd07e9314de05c54dc992ab76b95
Reviewed-on: https://go-review.googlesource.com/74010
Run-TryBot: Michael Munday <mike.munday@ibm.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Michael Munday <mike.munday@ibm.com>
Diffstat (limited to 'src/runtime/asm_s390x.s')
-rw-r--r-- | src/runtime/asm_s390x.s | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/runtime/asm_s390x.s b/src/runtime/asm_s390x.s index 42b9326607..487ed62306 100644 --- a/src/runtime/asm_s390x.s +++ b/src/runtime/asm_s390x.s @@ -915,7 +915,8 @@ TEXT runtime·sigreturn(SB),NOSPLIT,$0-0 RET TEXT ·publicationBarrier(SB),NOSPLIT|NOFRAME,$0-0 - SYNC + // Stores are already ordered on s390x, so this is just a + // compile barrier. RET TEXT runtime·cmpstring(SB),NOSPLIT|NOFRAME,$0-40 |