-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
GS/HW: Reduce number of copies for HDR #12254
base: master
Are you sure you want to change the base?
Conversation
ede16bb
to
5782cd6
Compare
5782cd6
to
8d0c2de
Compare
8d0c2de
to
0111e11
Compare
Awesome, thanks for the tests @crashGG ! Unfortunately the performance uplift is limited to a few games, but it's nice where it is :) HDR/COLCLIP has been a bit of a sticking point for performance over the years. |
Nice! By the way, does this affect native resolution? Also, does this affect software rendering? |
It affects all resolutions, only hardware renderers. |
It will depend on what you're limited by, however. if you're GPU limited, it will have the most significant effect, if you're CPU limited (on the GS thread) then you might notice a small performance increase.. But as mentioned it will be limited to what games are affected, mainly the ones listed, but there might be some other games affected, I just don't have GS dumps of them :) |
This is why I do not agree with this PR. According to the test data, if you want to experience high rendering (4K) under limited GPU conditions (core graphics), the best choice for RDNA3 GPU is still DX11. |
It's not really relevant to this PR, so please keep the discussion on-topic. |
OS: Linux 6.12; KDE 6.2.5; Manjaro Master: PR: NOTE: Seemingly because PR runs faster than master and because I was using F4, Sly got to the unwinnable boss fight at the end of the level faster, and so I had to run around in the same area for longer than on master, thus potentially skewing the results somewhat. But Sly getting to the boss fight so much faster was probably due in large part to the faster framerate (partially to getting better at completing the level in fast-forward). |
efaa566
to
6cdc778
Compare
38f48e0
to
7888fb5
Compare
e0e707f
to
0321db3
Compare
0321db3
to
242337e
Compare
THIS WILL ONLY AFFECT SPECIFIC GAMES!
Description of Changes
Tries to reduce the number of copies for HDR draws.
Rationale behind Changes
Master would copy the render target to a new texture, draw, then copy back, every single time, but a lot of games do a lot of separate draws in a row to the same target, so it's better to keep it in one place until the end of the chain. This PR will keep it HDR as long as possible before converting back, hugely reducing copies and render passes in some cases.
TL;DR: Makes some games go brr more
Suggested Testing Steps
Test the games listed below (sorry for the names)
Slightly Cherry Picked Reduction Results from Vulkan: