From: Cherry Mui Date: Fri, 14 Oct 2022 17:21:35 +0000 (-0400) Subject: runtime/internal/startlinetest: add NO_LOCAL_POINTERS macro to asm function X-Git-Tag: go1.20rc1~629 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=4e3d805a015153ca4e3382415c2474e4f954ddee;p=gostls13.git runtime/internal/startlinetest: add NO_LOCAL_POINTERS macro to asm function Should fix the mayMoreStack test. Change-Id: Iab99192629ab62ced81e95d756f2dd15f9b7e869 Reviewed-on: https://go-review.googlesource.com/c/go/+/443064 Auto-Submit: Austin Clements Reviewed-by: Michael Pratt Reviewed-by: Austin Clements Run-TryBot: Cherry Mui TryBot-Result: Gopher Robot --- diff --git a/src/runtime/internal/startlinetest/func_amd64.s b/src/runtime/internal/startlinetest/func_amd64.s index 6baeba28d4..ace5b34e70 100644 --- a/src/runtime/internal/startlinetest/func_amd64.s +++ b/src/runtime/internal/startlinetest/func_amd64.s @@ -2,6 +2,7 @@ // Use of this source code is governed by a BSD-style // license that can be found in the LICENSE file. +#include "funcdata.h" #include "textflag.h" // Assembly function for runtime_test.TestStartLineAsm. @@ -20,6 +21,7 @@ // callerStartLine since it is in a different package. TEXT ·AsmFunc(SB),NOSPLIT,$8-0 + NO_LOCAL_POINTERS MOVQ $0, AX // wantInlined CALL runtime_test·callerStartLine(SB) RET diff --git a/src/runtime/start_line_amd64_test.go b/src/runtime/start_line_amd64_test.go index c528adf3e2..57001e71de 100644 --- a/src/runtime/start_line_amd64_test.go +++ b/src/runtime/start_line_amd64_test.go @@ -13,7 +13,7 @@ import ( // is only tested on amd64 to avoid the need for a proliferation of per-arch // copies of this function. func TestStartLineAsm(t *testing.T) { - const wantLine = 22 + const wantLine = 23 got := startlinetest.AsmFunc() if got != wantLine { t.Errorf("start line got %d want %d", got, wantLine)