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

no getregs and setregs support in aarch64 #1419

Closed
sunc-cn opened this issue Apr 6, 2021 · 6 comments · Fixed by #2044
Closed

no getregs and setregs support in aarch64 #1419

sunc-cn opened this issue Apr 6, 2021 · 6 comments · Fixed by #2044

Comments

@sunc-cn
Copy link

sunc-cn commented Apr 6, 2021

when to support full setregs/getregs etc in aarch64

@asomers
Copy link
Member

asomers commented Apr 6, 2021

When somebody submits a patch for it.

@devnexen
Copy link
Contributor

will try to push the struct side of things in libc for next release at least.

@irgstg
Copy link

irgstg commented Apr 10, 2022

I have implemented some in a personal project I have.
I will open PR as soon as I can.

@sgleske-ias
Copy link

@asomers said:

When somebody submits a patch for it.

Please follow through on your comment with #1695 ; two years seems a bit past the period for respecting a contributor's time considering how you replied to this issue.

@SteveLauC
Copy link
Member

SteveLauC commented Mar 26, 2024

Well, it is unfortunate that PR didn't get reviewed for 2 years, and I am sorry about this, I plan to take a look at that PR this weekend.

Please follow through on your comment with #1695

Please don't blame asomser, it is not easy to maintain Nix, reviewing a PR can require several hours, and he was experiencing burnout at that time, as you can see from this comment.

@SteveLauC SteveLauC linked a pull request Mar 31, 2024 that will close this issue
@SteveLauC
Copy link
Member

This issue will be closed by #2044

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants