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

cmd/asm: no PRFM support for ARM64 platform #22932

Closed
mengzhuo opened this issue Nov 30, 2017 · 2 comments
Closed

cmd/asm: no PRFM support for ARM64 platform #22932

mengzhuo opened this issue Nov 30, 2017 · 2 comments
Labels
FeatureRequest FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@mengzhuo
Copy link
Contributor

What version of Go are you using (go version)?

go version devel +9a13f8e Tue Nov 28 06:47:50 2017 +0000 linux/arm64

Does this issue reproduce with the latest release?

yes

What operating system and processor architecture are you using (go env)?

GOARCH="arm64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOEXE=""
GOHOSTARCH="arm64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/root/go"
GORACE=""
GOROOT="/root/godev"
GOTMPDIR=""
GOTOOLDIR="/root/godev/pkg/tool/linux_arm64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build946453649=/tmp/go-build -gno-record-gcc-switches"

What did you do?

check cmd/asm/internal/asm/testdata/arm64enc.s
I found PRFM is at TODO stage

273:   //TODO PRFM 0x6400(R7), PSTL2STRM          // f300b2f9
274:   //TODO PRFM -215799(PC), PLIL2KEEP         // 2aa196d8

What did you expect to see?

PRFM works on arm64 platform

What did you see instead?

PRFM not supported

NOTE

I've already contacted Fannie Zhang who is a ARM employee and willing to import this inst for us :)

@bradfitz bradfitz changed the title cmd/asm no PRFM support for ARM64 platform cmd/asm: no PRFM support for ARM64 platform Nov 30, 2017
@bradfitz bradfitz added the NeedsFix The path to resolution is known, but the work has not been done. label Nov 30, 2017
@bradfitz bradfitz added this to the Unplanned milestone Nov 30, 2017
@zhangfannie
Copy link
Contributor

@mengzhuo the fix patch will be ready.

@gopherbot
Copy link

Change https://golang.org/cl/81675 mentions this issue: Add the Golang arm64 assembler support for instruction PRFM

@golang golang locked and limited conversation to collaborators Feb 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FeatureRequest FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

4 participants