Skip to content

Fix amdgpu & other issues #2000

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

Merged
merged 1 commit into from
Jan 2, 2025
Merged

Conversation

liberodark
Copy link
Contributor

@liberodark liberodark commented Dec 31, 2024

upstream some fixes from nixos package.

@squidbus
Copy link
Collaborator

Can you explain what these changes are for?

@tomboylover93
Copy link
Contributor

It likely refers to issue #1994, which is about shadPS4 failing to compile with GCC on one of the AMDGPU related files.

@squidbus
Copy link
Collaborator

Okay. I don't really understand the need for the swizzle init change, but for the DataFormat() change could you please change DataFormat() -> GetDataFmt() and NumberFormat() -> GetNumberFmt() to match what we have in other resources?

@liberodark liberodark force-pushed the main branch 2 times, most recently from f348c20 to de72ebf Compare January 1, 2025 00:46
@liberodark
Copy link
Contributor Author

liberodark commented Jan 1, 2025

In the CompMapping structure, we had initialization issues with brace initialization lists. The fix uses explicit member construction to avoid compilation errors.

Also this fix issue on GCC14

@liberodark liberodark force-pushed the main branch 3 times, most recently from 2e7b5fb to 71eeede Compare January 1, 2025 01:08
@raphaelthegreat raphaelthegreat merged commit 596f4cd into shadps4-emu:main Jan 2, 2025
10 checks passed
Xcedf added a commit to Xcedf/shadPS4 that referenced this pull request Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants