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

Enable HDR support in iOS. #15871

Merged
merged 2 commits into from
Nov 6, 2023

Conversation

warmenhoven
Copy link
Contributor

@warmenhoven warmenhoven commented Nov 5, 2023

This required a small tweak to MoltenVK; it was incorrectly not reporting that iOS/tvOS support the pixel format that RetroArch tries to use for HDR.

@hizzlekizzle
Copy link
Contributor

whoa, that's a big increase in the moltenvk filesize for tvOS. Up 30 MB vs iOS' 200 KB increase. Is that normal / in-line with expectations?

@warmenhoven
Copy link
Contributor Author

Thanks, didn't even notice that. It should not be as it should basically just be a recompile. Let me look into that.

@warmenhoven warmenhoven force-pushed the warmenhoven/pr/ios-hdr branch from 3364cd9 to 89002ce Compare November 5, 2023 23:37
@warmenhoven
Copy link
Contributor Author

The "external dependencies" components of moltenvk had somehow been set to create debug builds. Recompiling them as release builds fixed the size issue. Good spot, thanks!

@LibretroAdmin LibretroAdmin merged commit 7237b6a into libretro:master Nov 6, 2023
22 checks passed
@warmenhoven warmenhoven deleted the warmenhoven/pr/ios-hdr branch January 5, 2024 17:20
Sunderland93 pushed a commit to Sunderland93/RetroArch that referenced this pull request Dec 26, 2024
* Enable HDR support in iOS.

* Enable HDR support in tvOS.
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.

3 participants