Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Conversation

LloydNicholson
Copy link

This is my first time I have written c or c++, so please let me know what else I can do to make this code more robust or any refactors that are required.

Please let me know how I can improve the code. I am under no illusion that this not everything needed for this to work across different 9060 series (gfx1200/navi44) cards but I have it working on mine and that was my goal from the start anyway.

I honestly just wanted to have a better experience with a game I was enjoying and stop enjoying due to an update, so I fixed it to the best of my ability and maybe it can help to get this change into dev.

I did it to the point where I can run Indiana Jones and the Great Circle because it was bugging me that I couldn't and there we some serious performance issues around the save points and I knew AMDVLK could fix the issue.

I haven't done any other meaningful testing other than the fact that my ASRock Challenger 9060 XT 16GB works perfectly so far with AMDVLK in Indiana Jones instead of throwing a VK_ERROR_INCOMPATIBLE_DRIVER.

I added a PR to pal repo as well -> GPUOpen-Drivers/pal#109

@jinjianrong
Copy link
Member

@LloydNicholson Thanks for your contribution. However, the AMDVLK open source project is discontinued as announced here. You are welcome to contribute to RADV.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants