Skip to content

[AAPCS64] Remove ILP32 Support - #413

Open
smithp35 wants to merge 2 commits into
ARM-software:mainfrom
smithp35:ilp32aapcs64
Open

smithp35 wants to merge 2 commits into
ARM-software:mainfrom
smithp35:ilp32aapcs64

Conversation

@smithp35

Copy link
Copy Markdown
Contributor

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

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
Comment thread aapcs64/aapcs64.rst Outdated
Comment thread aapcs64/aapcs64.rst Outdated
Comment thread aapcs64/aapcs64.rst Outdated
- Removed ILP32 from definitions.
- Removed sentence mentioning ILP32 removal.
- Added ILP32 definition to change-log entry.
- Fixed two code-models and not three after ILP32 removal.

@smithp35 smithp35 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've uploaded a patch with review comments addressed. Will update the other docs to match.

Comment thread aapcs64/aapcs64.rst Outdated
Comment thread aapcs64/aapcs64.rst Outdated
Comment thread aapcs64/aapcs64.rst Outdated

@stuij stuij left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants