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

Don't defer EFB Copies to RAM - PMTTYD Issues #8766

Merged
merged 1 commit into from Apr 27, 2020
Merged

Don't defer EFB Copies to RAM - PMTTYD Issues #8766

merged 1 commit into from Apr 27, 2020

Conversation

teconmoon
Copy link
Contributor

This resolves a few issues with bounding box animations and others. Most noticably, it greatly reduces the bounding box slowdown seen on some NVIDIA cards and also fixes the odd overlay glitches when moving between rooms on the Excess Express.

This resolves a few issues with bounding box animations and others. Most noticably, it greatly reduces the bounding box slowdown seen on some NVIDIA cards and also fixes the odd overlay glitches when moving between rooms on the Excess Express.
@JMC47
Copy link
Contributor

JMC47 commented Apr 26, 2020

Sounds reasonable as a conclusion. This game does weird things with EFB2RAM, so the deferred copies could break things.

@mbc07
Copy link
Contributor

mbc07 commented Apr 26, 2020

Any chance of this also affecting Super Paper Mario?

@JMC47
Copy link
Contributor

JMC47 commented Apr 26, 2020

As both games use similar effects, I'd say probably.

@teconmoon
Copy link
Contributor Author

I can test SPM and post my results here, I don't recall having visual issues in my last Dolphin playthrough but not recent enough that I trust my memory.

@JMC47
Copy link
Contributor

JMC47 commented Apr 26, 2020

It's not reason not to merge this though.

@mbc07
Copy link
Contributor

mbc07 commented Apr 26, 2020

Definitely not a blocker, was just curious...

@leoetlino leoetlino merged commit b7ba72c into dolphin-emu:master Apr 27, 2020
@teconmoon teconmoon deleted the patch-1 branch April 27, 2020 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants