Skip to content

[BOLT][AArch64] Handle .plt.got section #71216

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

Merged
merged 1 commit into from
Nov 3, 2023
Merged

Conversation

yota9
Copy link
Member

@yota9 yota9 commented Nov 3, 2023

It seems that currently this section is only created by the mold linker if 2 conditions are met: 1. The PLT function was called directly. 2. The indirect access to PLT function was found (e.g. through ADRP relocation). Although mold created symbol for every plt entry I've removed them in yaml file to check that .plt.got was truly disassembled by bolt.

@yota9 yota9 added the BOLT label Nov 3, 2023
Copy link
Contributor

@aaupov aaupov left a comment

Choose a reason for hiding this comment

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

LGTM. I assume it fixes #70771? Please link this issue and run a spell checker on a commit message before merging.

@yota9
Copy link
Member Author

yota9 commented Nov 3, 2023

@aaupov Thanks. Ok, will do. No, I don't think so. I just new about this /possible/ problem with mold linker, it was found by @ElvinaYakubova some time ago, so I've decided to fix it first before checking the other stuff. For now it is more looks like yet another LongJmp problem, but didn't check it yet.

It seems that currently this section is only created by the mold linker
if 2 conditions are met: 1. The PLT function was called directly.
2. The indirect access to PLT function was found (e.g. through ADRP
relocation). Although mold created symbol for every plt entry I've
removed them in yaml file to check that .plt.got was truly disassembled
by bolt.
@yota9 yota9 merged commit 888742a into llvm:main Nov 3, 2023
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