Skip to content
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

JitArm64: Flush in the middle of lmw/stmw/mfcr #12352

Merged
merged 1 commit into from Dec 10, 2023

Conversation

JosJuice
Copy link
Member

@JosJuice JosJuice commented Dec 3, 2023

Normally we only flush registers right at the end of each PPC instruction. However, for PPC instructions that use a lot of registers one at a time, it's beneficial to do this flushing work in the middle of the instruction instead, reducing the risk of register starvation and improving pipelining.

@JosJuice
Copy link
Member Author

JosJuice commented Dec 3, 2023

@Kamielvdske Have you read the code when you press "approve"?

Normally we only flush registers right at the end of each PPC
instruction. However, for PPC instructions that use a lot of registers
one at a time, it's beneficial to do this flushing work in the middle
of the instruction instead, reducing the risk of register starvation
and improving pipelining.
@lioncash lioncash merged commit e2472e4 into dolphin-emu:master Dec 10, 2023
11 checks passed
@JosJuice JosJuice deleted the jitarm64-flush-in-long-inst branch December 10, 2023 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants