-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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: Allow data references to labels in assembly (support for jump tables) #10870
Comments
The usual way to implement jump tables on amd64 is something like tableStart: and then the code that uses the table loads the word into a register, adds tableStart, and jumps to the register. |
A related compiler request: #5496.
|
In hand-written asm you can do jump tables, though slighly less efficient than you may want (requires 1 more instruction for dispatch). In short:
(full read, compete example included: go-asm-dispatch-tables) But please, measure result performance carefully. Good luck. |
If this solution is acceptable, this issue should probably be closed. |
Thanks for the write up! Would "real" labels as data support needed for #5496? |
@martisch : Probably not. The compiler "assembles" its own instructions, so the assembler proper would need no changes. We might need some common support in the obj library. It would be weird for the compiler to generate code that the assembler can't, though. |
I've noticed a couple comments in Go assembly routines "TODO replace this with a jump table in the future" and when I tried implementing an assembly routine with a jump table I realized why it hasn't been tried yet. The assembler does not support using labels in data, so there's no easy way to construct the jump table. My incomplete understanding of x64 architecture is that jumps are signed 32 bit offsets from the instruction pointer, which seems like it would make it difficult to implement. Maybe there's also a way to do a jump to a 64bit virtual address, I don't know.
Anyway if it's not too difficult, this would be a nice to have.
The text was updated successfully, but these errors were encountered: