Skip to content

Fix ios aot dotnet crash#106511

Closed
TCROC wants to merge 1 commit into
godotengine:masterfrom
Lange-Studios:fix-ios-dotnet-crash
Closed

Fix ios aot dotnet crash#106511
TCROC wants to merge 1 commit into
godotengine:masterfrom
Lange-Studios:fix-ios-dotnet-crash

Conversation

@TCROC
Copy link
Copy Markdown
Contributor

@TCROC TCROC commented May 16, 2025

Fixes the ios portion of:

#96072

I noticed that this user also reported they are having crashes on their windows machine. I'll see if I can reproduce it on ours as well. Currently I've only fixed it for ios. Granted I'm not super happy with this fix as the solution is to just disable optimizations entirely. Maybe a more fine grained approach of disabling optimizations per godot interface method is better.

@TCROC TCROC requested a review from a team as a code owner May 16, 2025 22:13
@Chaosus Chaosus added this to the 4.x milestone May 17, 2025
@TCROC
Copy link
Copy Markdown
Contributor Author

TCROC commented May 19, 2025

NOTE: I don't like this PR. Although I do think we should maybe document or list out warnings surrounding having "optimize" enabled to true on ios until we solve the other issue. But for now I'm going to close this PR and likely revert it in my own fork in favor of adding this to my root csproj:

    <Optimize Condition="'$(GodotTargetPlatform)' == 'ios'">false</Optimize>

@TCROC TCROC closed this May 19, 2025
@akien-mga akien-mga removed this from the 4.x milestone May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants