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/arch/arm/armasm: plan9 register list argument print incorrectly #19142

Closed
williamweixiao opened this issue Feb 17, 2017 · 3 comments
Closed

x/arch/arm/armasm: plan9 register list argument print incorrectly #19142

williamweixiao opened this issue Feb 17, 2017 · 3 comments
Milestone

Comments

@williamweixiao
Copy link
Member

@williamweixiao williamweixiao commented Feb 17, 2017

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

golang.org/x/arch/arm

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

Ubuntu 16.04.1 LTS and ARM64

What did you do?

Disassemble following instruction
ed003be9

What did you expect to see?

LDMDB [R0,R2-R3,R5-R7], R11!

What did you see instead?

LDMDB [R5-R7], R11!

@gopherbot
Copy link

@gopherbot gopherbot commented Feb 17, 2017

CL https://golang.org/cl/37172 mentions this issue.

Loading

@williamweixiao
Copy link
Member Author

@williamweixiao williamweixiao commented Feb 21, 2017

@rsc any comments?

Loading

@bradfitz bradfitz added this to the Go1.9 milestone Mar 21, 2017
@gopherbot gopherbot closed this in 58ea1a1 Mar 23, 2017
gopherbot pushed a commit that referenced this issue Apr 20, 2017
Copied by hand.

Update #17410
Update #19142
Fixes #19986

Change-Id: I21d16d254161c75466b31c670f3b2c8c463abd66
Reviewed-on: https://go-review.googlesource.com/41205
Run-TryBot: Keith Randall <khr@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
@gopherbot
Copy link

@gopherbot gopherbot commented Jul 27, 2017

Change https://golang.org/cl/49530 mentions this issue: cmd/vendor/golang.org/x/arch: pull updates from x repo

Loading

gopherbot pushed a commit that referenced this issue Aug 16, 2017
Vendor from golang.org/x/arch (commit f185940).

Implements #19157

Updates #12840
Updates #20762
Updates #20897
Updates #20096
Updates #20766
Updates #20752
Updates #20096
Updates #19142

Change-Id: Idefb8ba2c355dc07f3b9e8dcf5f00173256a0f0f
Reviewed-on: https://go-review.googlesource.com/49530
Reviewed-by: Cherry Zhang <cherryyz@google.com>
@golang golang locked and limited conversation to collaborators Jul 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants