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.

Sign up for GitHub

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

Jump to bottom

math: Sqrt not intrinsified on mips64 #24006

Closed
ALTree opened this issue Feb 21, 2018 · 1 comment
Closed

math: Sqrt not intrinsified on mips64 #24006

ALTree opened this issue Feb 21, 2018 · 1 comment
Labels
FrozenDueToAge Performance

Comments

@ALTree
Copy link
Member

ALTree commented Feb 21, 2018 •
edited 

$ go version
go version go1.10 linux/amd64

func f(x float64) float64 {
	return math.Sqrt(x)
}

GOARCH=mips:

0x0004 00004 (test.go:5)	MOVF	"".x+4(FP), F0
0x0008 00008 (test.go:6)	SQRTD	F0, F0

but with GOARCH=mips64:

0x002c 00044 (test.go:5)	MOVD	"".x(FP), F0
0x0030 00048 (test.go:6)	MOVD	F0, 8(R29)
0x0034 00052 (test.go:6)	CALL	math.Sqrt(SB)
@gopherbot
Copy link

Change https://golang.org/cl/95916 mentions this issue: cmd/compile: intrinsify math.sqrt on mips64

@ALTree ALTree changed the title math.Sqrt: not intrinsified on mips64 math: Sqrt not intrinsified on mips64 Feb 21, 2018
@golang golang locked and limited conversation to collaborators Feb 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge Performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ALTree @gopherbot