]> Cypherpunks repositories - gostls13.git/commitdiff
runtime/internal/startlinetest: add NO_LOCAL_POINTERS macro to asm function
authorCherry Mui <cherryyz@google.com>
Fri, 14 Oct 2022 17:21:35 +0000 (13:21 -0400)
committerGopher Robot <gobot@golang.org>
Fri, 14 Oct 2022 18:33:34 +0000 (18:33 +0000)
Should fix the mayMoreStack test.

Change-Id: Iab99192629ab62ced81e95d756f2dd15f9b7e869
Reviewed-on: https://go-review.googlesource.com/c/go/+/443064
Auto-Submit: Austin Clements <austin@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Austin Clements <austin@google.com>
Run-TryBot: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

src/runtime/internal/startlinetest/func_amd64.s
src/runtime/start_line_amd64_test.go

index 6baeba28d4785c36fffc824dc54254e4a5ac8e8f..ace5b34e70f833efa4533a90b62536054128ec8e 100644 (file)
@@ -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<ABIInternal>(SB),NOSPLIT,$8-0
+       NO_LOCAL_POINTERS
        MOVQ    $0, AX // wantInlined
        CALL    runtime_test·callerStartLine<ABIInternal>(SB)
        RET
index c528adf3e240b9ad22007baf0d7c8e8e75fc9e58..57001e71de92759cd8492ffc6487f8d7785411b7 100644 (file)
@@ -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)