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

SPARC syscall6 may truncate the last parameter #8712

Open
LemonBoy opened this issue May 8, 2021 · 4 comments
Open

SPARC syscall6 may truncate the last parameter #8712

LemonBoy opened this issue May 8, 2021 · 4 comments
Labels
arch-sparc 32-bit and 64-bit SPARC backend-llvm The LLVM backend outputs an LLVM IR Module. bug Observed behavior contradicts documented or intended behavior upstream An issue with a third party project that Zig uses.
Milestone
0.13.0

Comments

@LemonBoy
Copy link
Contributor

LemonBoy commented May 8, 2021

Mostly as PSA so that I won't forget, the LLVM bug is fixed by this patch that's still pending approval :(

cc @koachan
@koachan
Copy link
Contributor

koachan commented May 9, 2021

Just checked it, with the patch SPARCv9 syscall6 now omits the correct ldx to load the last parameter. Thanks!

@andrewrk andrewrk added arch-sparc 32-bit and 64-bit SPARC bug Observed behavior contradicts documented or intended behavior upstream An issue with a third party project that Zig uses. labels May 19, 2021
@andrewrk andrewrk added this to the 0.9.0 milestone May 19, 2021
@andrewrk andrewrk added the backend-llvm The LLVM backend outputs an LLVM IR Module. label May 19, 2021
@andrewrk
Copy link
Member

Looks like the patch still did not land upstream.

@andrewrk andrewrk modified the milestones: 0.9.0, 0.10.0 Nov 20, 2021
@andrewrk andrewrk modified the milestones: 0.10.0, 0.11.0 Apr 16, 2022
@koachan
Copy link
Contributor

koachan commented Sep 15, 2022

By the way, the patch seems to be merged in LLVM 15, so I think we can close this issue?

@nektro
Copy link
Contributor

nektro commented Sep 15, 2022

can confirm

image

@andrewrk andrewrk modified the milestones: 0.11.0, 0.12.0 Jun 19, 2023
@RodrigoDornelles RodrigoDornelles mentioned this issue Oct 20, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
arch-sparc 32-bit and 64-bit SPARC backend-llvm The LLVM backend outputs an LLVM IR Module. bug Observed behavior contradicts documented or intended behavior upstream An issue with a third party project that Zig uses.
Projects
None yet
Milestone
0.13.0
Development

No branches or pull requests
4 participants
@LemonBoy @andrewrk @nektro @koachan