Skip to content

[LA64_DYNAREC] Added host_call to arch state - #4340

Merged
ptitSeb merged 2 commits into
ptitSeb:mainfrom
ksco:host-call-arch
Sep 1, 2026
Merged

[LA64_DYNAREC] Added host_call to arch state#4340
ptitSeb merged 2 commits into
ptitSeb:mainfrom
ksco:host-call-arch

Conversation

@ksco

@ksco ksco commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

This fixed box64-bash's unstablility on LoongArch.

Comment thread src/libtools/signals.c
@ptitSeb

ptitSeb commented Sep 1, 2026

Copy link
Copy Markdown
Owner

LGTM now. I suppose ARM64 doesn't needs that for now, as RSP is not clobbered during native call. But RV64 (and PPC64LE) might need something similar at some point.

@ptitSeb
ptitSeb merged commit 3f74c74 into ptitSeb:main Sep 1, 2026
28 checks passed
@xiangzhai

xiangzhai commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Hi,

But RV64 (and PPC64LE) might need something similar at some point.

RV64 might doesn't need. Because RV64 RSP mapping to x9 (s1), Saved register, not Function argument.
There is uintptr_t xSPSave in the x64emu_t for RV64 and PPC64LE. But LA64 xSavedSP mapping to $r22 ($fp), otherwise RSP mapped to $r22 instead.
But LA64 mapping to $r11 (a7), Function argument.
Do I need to refactor the la64 mapping?

Thanks,
Leslie Zhai

@ksco

ksco commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

It's not just RSP.

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