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

cmd/asm: ARM assembler should reject STREX R1, (R0), R1 #22268

Closed
ianlancetaylor opened this issue Oct 14, 2017 · 2 comments
Closed

cmd/asm: ARM assembler should reject STREX R1, (R0), R1 #22268

ianlancetaylor opened this issue Oct 14, 2017 · 2 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone
Go1.10

Comments

@ianlancetaylor
Copy link
Contributor

The ARM architecture manuals say that STREX does not permit the same register to be used as both the value to be stored and the place to record whether the store succeeded. The assembler should reject this instruction.
@ianlancetaylor ianlancetaylor added the NeedsFix The path to resolution is known, but the work has not been done. label Oct 14, 2017
@ianlancetaylor ianlancetaylor added this to the Go1.10 milestone Oct 14, 2017
@ianlancetaylor
Copy link
Contributor Author

CC @cherrymui

@cherrymui cherrymui self-assigned this Oct 16, 2017
@gopherbot
Copy link

Change https://golang.org/cl/71190 mentions this issue: cmd/asm: reject STREX with same source and destination register on ARM

@golang golang locked and limited conversation to collaborators Oct 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Milestone
Go1.10
Development

No branches or pull requests
3 participants
@ianlancetaylor @gopherbot @cherrymui