-
Notifications
You must be signed in to change notification settings - Fork 713
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
Correct usage of x86 directCallRequiresTrampoline() API #15024
Conversation
0xdaryl
commented
May 6, 2022
- Correct usage of x86 directCallRequiresTrampoline() API
- Replace Z NEEDS_TRAMPOLINE macro with function call
Calls to `directCallRequiresTrampoline()` should be passed the address of the start of the call instruction. Failing to do so may lead to incorrect branch displacements being computed. Signed-off-by: Daryl Maier <maier@ca.ibm.com>
There is not really a benefit to hiding the call to `directCallRequiresTrampoline()` behind a macro. Call the function directly. Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Depends eclipse/omr#6503 |
Jenkins test sanity xlinux,win,osx,zlinux,zos jdk17 depends eclipse/omr#6503 Edit: For the record, all builds and tests succeeded. |
Jenkins test sanity all jdk8,jdk11 depends eclipse/omr#6503 |
@ymanton : would you mind reviewing this and its dependent PR in OMR (eclipse/omr#6503) please? There is a common codegen API @JamesKingdon FYI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of questions.