mirror of
https://github.com/golang/go
synced 2024-11-19 17:04:41 -07:00
87e48c5afd
Since barrier-less memclr is only safe in very narrow circumstances, this commit renames memclr to avoid accidentally calling memclr on typed memory. This can cause subtle, non-deterministic bugs, so it's worth some effort to prevent. In the near term, this will also prevent bugs creeping in from any concurrent CLs that add calls to memclr; if this happens, whichever patch hits master second will fail to compile. This also adds the other new memclr variants to the compiler's builtin.go to minimize the churn on that binary blob. We'll use these in future commits. Updates #17503. Change-Id: I00eead049f5bd35ca107ea525966831f3d1ed9ca Reviewed-on: https://go-review.googlesource.com/31369 Reviewed-by: Keith Randall <khr@golang.org> Reviewed-by: Rick Hudson <rlh@golang.org>
38 lines
813 B
ArmAsm
38 lines
813 B
ArmAsm
// Copyright 2014 The Go Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
#include "textflag.h"
|
|
|
|
// void runtime·memclrNoHeapPointers(void*, uintptr)
|
|
TEXT runtime·memclrNoHeapPointers(SB),NOSPLIT,$0-16
|
|
MOVD ptr+0(FP), R3
|
|
MOVD n+8(FP), R4
|
|
// TODO(mwhudson): this is written this way to avoid tickling
|
|
// warnings from addpool when written as AND $7, R4, R6 (see
|
|
// https://golang.org/issue/12708)
|
|
AND $~7, R4, R5 // R5 is N&~7
|
|
SUB R5, R4, R6 // R6 is N&7
|
|
|
|
CMP $0, R5
|
|
BEQ nowords
|
|
|
|
ADD R3, R5, R5
|
|
|
|
wordloop: // TODO: Optimize for unaligned ptr.
|
|
MOVD.P $0, 8(R3)
|
|
CMP R3, R5
|
|
BNE wordloop
|
|
nowords:
|
|
CMP $0, R6
|
|
BEQ done
|
|
|
|
ADD R3, R6, R6
|
|
|
|
byteloop:
|
|
MOVBU.P $0, 1(R3)
|
|
CMP R3, R6
|
|
BNE byteloop
|
|
done:
|
|
RET
|