Skip to content

Conversation

@blueskythlikesclouds
Copy link
Contributor

@blueskythlikesclouds blueskythlikesclouds commented Oct 22, 2025

Fixes #111869.

Question: I figure there's no way for the OS singleton to be nullptr in this function?

@Calinou Calinou added platform:windows cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release labels Oct 22, 2025
@Calinou Calinou added this to the 4.6 milestone Oct 22, 2025
@Calinou Calinou added the cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release label Oct 22, 2025
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would modify main/main.cpp to change the message in the --generate-spirv-debug-info help:

Generate SPIR-V debug information (Vulkan only). This allows source-level shader debugging with RenderDoc.

@stuartcarnie Does it work when using Metal?

@blueskythlikesclouds blueskythlikesclouds force-pushed the disable-spirv-debug-info-on-d3d12 branch from 2ab782a to a62b194 Compare October 23, 2025 08:17
@blueskythlikesclouds blueskythlikesclouds requested a review from a team as a code owner October 23, 2025 08:17
@clayjohn clayjohn removed the cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release label Oct 23, 2025
@clayjohn clayjohn requested a review from Calinou October 23, 2025 17:38
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

I've tested this on macOS with Metal and the argument seems to behave fine, so it's probably fine to leave it enabled there.

@Repiteo Repiteo merged commit 87b46c6 into godotengine:master Oct 23, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Oct 23, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release platform:windows topic:rendering

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Godot editor crashes immediately when using D3D12 and with --generate-spirv-debug-info enabled

4 participants