-
Notifications
You must be signed in to change notification settings - Fork 437
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
Revert "Vulkan: Feedback loop detection and barriers (#7226)" #264
base: master
Are you sure you want to change the base?
Conversation
570fdd4
to
fd84e07
Compare
Thought I should add my two cents to this. This is not a good idea, it would be better to use performance and best practice layer to fix this. |
There's a reason why I haven't merged it yet. We're figuring out a way to keep the behavior when it needs it. |
Now that the sparse-jit PR has been merged, would you consider rebasing this PR on the latest Canary release? I didn't see any benefits from either sparse jit nor this Vulkan barrier revert before since I have a weak CPU, but I am curious to see if there will be benefits from having both of them at once. |
cb46ddf
to
e879a0c
Compare
Done. |
d781691
to
939fabe
Compare
The City in Xenoblade went from 40 FPS on both the sparse jit build and barrier revert build individually, to 50 - 55 FPS when both are merged together in this pull request. I guess all that we need to finish it is some way of detecting RDNA3 GPUs to enable vulkan barrier detection for them specifically. |
This reverts commit ca59c3f.
939fabe
to
f702cf2
Compare
This reverts commit ca59c3f.
Prevents slowdowns in Paper Mario: The Thousand Year Door, the Xenoblade games, and others. Feel free to test and let me know if you get speedups in games where it would normally slow down.
There are known issues with 7000 series AMD cards, so an AMD-only fix would be ideal.