Repository navigation
[AAELF64] Remove ILP32 - #414
Merged
Merged
Conversation
stuij
requested changes
Sep 11, 2026
smithp35
commented
Sep 11, 2026
smithp35
left a comment
smithp35
left a comment
Contributor
Author
There was a problem hiding this comment.
Thanks for the review, will send another revision shortly.
stuij
reviewed
Sep 17, 2026
ILP32 support for ELF platforms in the documents has been in perpetual Beta, with new additions to the ABI not considering ILP32. Now that the one complete implementation in GCC has been deprecated (https://gcc.gnu.org/gcc-15/changes.html) we will be removing ILP32 from the ABI documentation. Part of ARM-software#369
- Remove ILP32 note in favour of changelog. - Remove ILP32 definition, inline the text in changelog.
* Fix typo. * remove dangling list. * One ELF class. * R_AARCH64_ABS64 is singular.
The disambiguation between ilp32 and ilp64 relocation codes is no longer necessary. The distinction between static and dynamic relocations works without it.
All that remains in the section is the distinction between static and dynamic relocations. Rename to ranges.
smithp35
force-pushed
the
ilp32aaelf64
branch
from
September 17, 2026 12:20
61aa07a to
c1aa8ed
Compare
philgee-oss
reviewed
Oct 12, 2026
|
|
||
| ``R_AARCH64_ABS64`` may only appear in a well-formed executable or dynamic shared object. Note that this relocation is both a static and a dynamic relocation. | ||
|
|
||
| ``R_AARCH64_COPY`` may only appear in executable ELF files where e\_type is set to ``ET_EXEC``. The effect is to cause the dynamic linker to locate the target symbol in a shared library object and then to copy the number of bytes specified by its ``st_size`` field to the place. The address of the place is then used to pre-empt all other references to the specified symbol. It is an error if the storage space allocated in the executable is insufficient to hold the full copy of the symbol. If the object being copied contains dynamic relocations then the effect must be as if those relocations were performed before the copy was made. |
There was a problem hiding this comment.
Remove extra spacing from
"to cause"
"of bytes"
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ILP32 support for ELF platforms in the documents has been in perpetual Beta, with new additions to the ABI not considering ILP32. Now that the one complete implementation in GCC has been deprecated (https://gcc.gnu.org/gcc-15/changes.html) we will be removing ILP32 from the ABI documentation.
Part of #369