Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/tools/go/analysis/passes/asmdecl: TestVetStdlib failures on loong64 #68041

Closed
abner-chenc opened this issue Jun 18, 2024 · 2 comments
Closed
Labels
Tools This label describes issues relating to any tools in the x/tools repository.
Milestone

Comments

@abner-chenc
Copy link
Contributor

Go version

go version devel go1.23-45446c867a Tue Jun 18 01:40:18 2024 +0000 linux/loong64

Output of go env in your module/workspace:

GO111MODULE=''
GOARCH='loong64'
GOBIN=''
GOCACHE='/tmp/abner/gocache'
GOENV='/home/data/chenguoqi/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='loong64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/data/chenguoqi/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/data/chenguoqi/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/home/data/chenguoqi/golang/go-upstream'
GOSUMDB='sum.golang.org'
GOTMPDIR='/tmp/abner/gotmp'
GOTOOLCHAIN='auto'
GOTOOLDIR='/home/data/chenguoqi/golang/go-upstream/pkg/tool/linux_loong64'
GOVCS=''
GOVERSION='devel go1.23-45446c867a Tue Jun 18 01:40:18 2024 +0000'
GODEBUG=''
GOTELEMETRY='local'
GOTELEMETRYDIR='/home/data/chenguoqi/.config/go/telemetry'
GCCGO='gccgo'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/home/data/chenguoqi/golang/tools/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -mabi=lp64d -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/abner/gotmp/go-build414030735=/tmp/go-build -gno-record-gcc-switches'

What did you do?

git clone https://go.googlesource.com/tools && cd tools && go test -v ./...

What did you see happen?

ok golang.org/x/tools/go/analysis/passes/unusedresult 5.221s
ok golang.org/x/tools/go/analysis/passes/unusedwrite 1.254s
ok golang.org/x/tools/go/analysis/passes/usesgenerics 1.082s
--- FAIL: TestVetStdlib (9.11s)
vet_std_test.go:101: go vet std failed (exit status 1):
# internal/bytealg
../../../../go-upstream/src/internal/bytealg/indexbyte_loong64.s:26:1: [loong64] IndexByte: RET without writing to result register
../../../../go-upstream/src/internal/bytealg/indexbyte_loong64.s:30:1: [loong64] IndexByte: RET without writing to result register
../../../../go-upstream/src/internal/bytealg/indexbyte_loong64.s:48:1: [loong64] IndexByteString: RET without writing to result register
../../../../go-upstream/src/internal/bytealg/indexbyte_loong64.s:52:1: [loong64] IndexByteString: RET without writing to result register
FAIL
FAIL golang.org/x/tools/go/analysis/unitchecker 27.697s
ok golang.org/x/tools/go/ast/astutil 0.163s
ok golang.org/x/tools/go/ast/inspector 0.550s
ok golang.org/x/tools/go/buildutil 3.829s
ok golang.org/x/tools/go/callgraph 0.029s [no tests to run]

What did you expect to see?

all test pass

@gopherbot gopherbot added the Tools This label describes issues relating to any tools in the x/tools repository. label Jun 18, 2024
@gopherbot gopherbot added this to the Unreleased milestone Jun 18, 2024
@abner-chenc abner-chenc changed the title x/tools/go/analysis/passes/asmdecl: TestVetStdlib failures on linux/loong64 x/tools/go/analysis/passes/asmdecl: TestVetStdlib failures on loong64 Jun 18, 2024
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/593275 mentions this issue: go/analysis/passes/asmdecl: define register-ABI result registers for loong64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Development

No branches or pull requests

3 participants