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

HLE_OS: Don't modify emulated state in printf hooks. #10925

Merged
merged 1 commit into from
Jul 30, 2022

Conversation

AdmiralCurtiss
Copy link
Contributor

@AdmiralCurtiss AdmiralCurtiss commented Jul 30, 2022

The hooks that call these are Start hooks, so the real PPC function is called afterwards and takes care of any state changes.

@Pokechu22
Copy link
Contributor

What about HLE_write_console, HLE_LogDPrint, and HLE_LogFPrint? Those also have the same NPC = LR line and are start hooks, though they're probably less frequently used.

@AdmiralCurtiss
Copy link
Contributor Author

Yeah, those should probably be removed too.

Those fifoci diffs are curious...

@Pokechu22
Copy link
Contributor

I'm pretty sure the fifoci diffs are a separate issue related the radeon builder using its Intel GPU instead of its AMD GPU for some reason; they're not directly caused by your change. (A similar thing happened a few days ago.)

Copy link
Contributor

@sepalani sepalani left a comment

Choose a reason for hiding this comment

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

These make sense since they aren't replace hook anymore. LGTM.

@JosJuice JosJuice merged commit ecc4bc5 into dolphin-emu:master Jul 30, 2022
@AdmiralCurtiss AdmiralCurtiss deleted the ppc-state-hle-print branch July 30, 2022 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants