Skip to content

static trampoline for LoongArch#723

Merged
atgreen merged 1 commit intolibffi:masterfrom
xry111:xry111/la-static-tramp
Jul 21, 2022
Merged

static trampoline for LoongArch#723
atgreen merged 1 commit intolibffi:masterfrom
xry111:xry111/la-static-tramp

Conversation

@xry111
Copy link
Contributor

@xry111 xry111 commented Jul 10, 2022

For the benefit and technical details of static trampoline, see
#624. As a new architecture, let's
be "safer" from the start.

The change survived libffi testsuite on loongarch64-linux-gnu.

For the benefit and technical details of static trampoline, see
libffi#624.  As a new architecture, let's
be "safer" from the start.

The change survived libffi testsuite on loongarch64-linux-gnu.
Copy link
Contributor Author

@xry111 xry111 left a comment

Choose a reason for hiding this comment

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

@xen0n Please review.
@Panxuefeng-loongson Please test if this change broke your downstream works.

.size ffi_call_asm, .-ffi_call_asm


/* ffi_closure_asm. Expects address of the passed-in ffi_closure in t1.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: this is a document typo, which cost me half an hour to debug some "bus error".

@Panxuefeng-loongson
Copy link
Contributor

@xen0n Please review. @Panxuefeng-loongson Please test if this change broke your downstream works.

Sorry for not replying to you in time. Currently, all I can use are loongnix machines. So, there is no way to verify on the new world machine. If all libffi tests pass, it will not have a very big impact on applications such as jna. Thanks

@atgreen atgreen merged commit 5a4774c into libffi:master Jul 21, 2022
@xen0n
Copy link

xen0n commented Jul 22, 2022

Sorry for the late reply, I briefly looked at the code and it seemed great. I hoped to integrate this into Gentoo for wide testing but $DAY_JOB kicked in (along with backlogged review requests for several other LoongArch porting efforts), so I never finished the review. Nice to see it merged though, I'll get around to testing this later.

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.

4 participants