-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Some additions to std.elf
addressing #19830, plus some zld improvements
#21573
Conversation
These have no callers outside std.elf. Even if the standard library should provide functions like these, std.elf is probably not the place, given how general they are.
/// NVIDIA CUDA architecture | ||
CUDA = 51, | ||
/// AMD HSA Runtime | ||
AMDGPU_HSA = 64, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For OSABI values > 64, they become dependent on the machine header, and so can overlap. Could we maybe, move them to be decls with multiple having the same value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The pattern we usually use in situations like this is to have the most common name be the proper enum tag and the uncommon ones be decls defined as that enum tag. That seems reasonable to do here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can agree to that :)
Closes #19830.