Skip to content

[baabielf64] AArch64 ELF Conventions for Binary Analysis.#349

Merged
smithp35 merged 3 commits intoARM-software:mainfrom
ilinpv:main
Sep 29, 2025
Merged

[baabielf64] AArch64 ELF Conventions for Binary Analysis.#349
smithp35 merged 3 commits intoARM-software:mainfrom
ilinpv:main

Conversation

@ilinpv
Copy link
Copy Markdown
Contributor

@ilinpv ilinpv commented Sep 10, 2025

Added new alpha draft document to specify code patterns and conventions for binaries intended to improve recognition by binary analysis tools. AArch64 veneers (aka thunks, stubs or trampolines) were documented there reflecting the current implementations in ld.bfd, ld.lld, mold and go linker.

Added new alpha draft document to specify code patterns and
conventions for binaries intended to improve recognition by binary
analysis tools. AArch64 veneers (aka thunks, stubs or trampolines)
were documented there reflecting the current implementations in
ld.bfd, ld.lld, mold and go linker.
@ilinpv
Copy link
Copy Markdown
Contributor Author

ilinpv commented Sep 10, 2025

As suggested, I created a separate document to support binary analysis tools, addressed comments from the previous PR #333 ( tagging reviewers and commenters from the prior discussion to this PR @smithp35 @rui314 @maksfb @Wilco1 @MaskRay ), documented the new veneers and reflected the current state of what the major linkers can generate. Proposals to improve the veneers design and implementation are out of scope for this PR and can be considered separately.

Copy link
Copy Markdown
Contributor

@smithp35 smithp35 left a comment

Choose a reason for hiding this comment

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

Thanks for splitting this out.

Comment thread baabielf64/baabielf64.rst Outdated
Comment thread baabielf64/baabielf64.rst Outdated
Comment thread baabielf64/baabielf64.rst Outdated
Comment thread baabielf64/baabielf64.rst Outdated
Comment thread baabielf64/baabielf64.rst Outdated
@ilinpv
Copy link
Copy Markdown
Contributor Author

ilinpv commented Sep 16, 2025

Thank you @smithp35 for the review and helpful comments, they have been addressed.

Copy link
Copy Markdown
Contributor

@smithp35 smithp35 left a comment

Choose a reason for hiding this comment

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

Thanks for the updates. I don't have any more comments at the moment. I've set approved as I think that this is good enough for the intended Alpha status.

Copy link
Copy Markdown
Contributor

@MaskRay MaskRay left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread baabielf64/baabielf64.rst Outdated
Comment thread baabielf64/baabielf64.rst Outdated
@ilinpv
Copy link
Copy Markdown
Contributor Author

ilinpv commented Sep 23, 2025

Thanks @MaskRay, I appreciate the review. Unless there are more comments, I would merge it at the end of the week.

@smithp35 smithp35 merged commit 4a3e394 into ARM-software:main Sep 29, 2025
1 check passed
@smithp35
Copy link
Copy Markdown
Contributor

Document has been approved, merging for @ilinpv erging for @ilinpv

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 this pull request may close these issues.

3 participants