Skip to content

render: Added Xbox DXIL for gpu backend#15121

Merged
flibitijibibo merged 1 commit into
libsdl-org:mainfrom
flibitijibibo:xbox_gpu_render
Mar 2, 2026
Merged

render: Added Xbox DXIL for gpu backend#15121
flibitijibibo merged 1 commit into
libsdl-org:mainfrom
flibitijibibo:xbox_gpu_render

Conversation

@flibitijibibo

Copy link
Copy Markdown
Collaborator

This gets Xbox running with render_gpu, but I got the root signatures all wrong - will need someone more familiar with D3D12 to get working.

@Starkium

Copy link
Copy Markdown
Contributor

this is awesome, but does it matter anymore? xbox is kind of dead. Might be good for the modding / homebrew scene if they can unlock the Xboxes to use more than UWP. Maybe that's already the case, I'm a few years behind.

@flibitijibibo
flibitijibibo marked this pull request as ready for review March 1, 2026 23:37
@flibitijibibo
flibitijibibo merged commit f2c4c66 into libsdl-org:main Mar 2, 2026
45 checks passed
@flibitijibibo
flibitijibibo deleted the xbox_gpu_render branch March 2, 2026 00:09
@TheSpydog

Copy link
Copy Markdown
Collaborator

Sorry, it looks like this broke the GDK build, so we should revert this change. Xbox precompilation fails without root signature definitions, but the good news is that the original dxil.h just works, so we don't actually need any changes to what's already there for Windows!

@flibitijibibo

Copy link
Copy Markdown
Collaborator Author

Sure, we can revert this... how does the PC DXIL work though? Is this something cert would complain about?

@TheSpydog

Copy link
Copy Markdown
Collaborator

Nope, it should be totally fine. DXIL isn't directly consumed, it's just an intermediate format. Xbox can compile shaders at runtime from both HLSL and DXIL source (which is how ShaderCross and GLon12 work). It's just not as optimal as full precompilation, which allows the Xbox to skip the DXIL -> machine code step at runtime.

@flibitijibibo

Copy link
Copy Markdown
Collaborator Author

This has been reverted in main and release-3.4.x.

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