-
Notifications
You must be signed in to change notification settings - Fork 163
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
Add CREL support #439
Comments
I don't think we should be adopting draft gABI proposals, and if and when it becomes an official part of the gABI there's nothing for us to say because our psABI only mentions (AFAIK) the relocations, leaving the representation to the gABI. |
Thanks for the interest and support for CREL. RELLEB was an earlier, abandonded name. As I mentioned in the RELR issue: In practice, as long as there's significant benefit and broad support from GNU, LLVM, and a sizeable user base, the generic ABI inclusion becomes less relevant. Based on my discussions, Arm, RISC-V, and s390x architectures seem open to including SHT_CREL in their generic ABI ranges. I agree replicating the existing specification of CREL is unnecessary. In practice, as long as we choose a section code, achieve agreement from GNU and LLVM, and commit to its stability, formal approval from the generic ABI might not be essential. (Let's be honest, updates beyond constants (new machine code, ELFCOMPRESS_ZSTD) haven't happened for the generic ABI for many years.) RISC-V's buy-in is crucial for aligning GNU and LLVM. Thanks again. |
Yeah, I generally agree, but it's kinda chicken egg issue, CREL need implementation and support from targets, so that can become official gABI, the goal of creating this issue is kinda explicitly to say we are interesting on this, then @MaskRay can take this as a more concrete reference/evidence to push that forward :) |
CREL is a LEB128-based relocation format, which can be significantly reduce the object size and also remove the limitation of the number of the relocation types.
Related discussions:
The text was updated successfully, but these errors were encountered: