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

32-bit builds of Godot do not work with LLVM MinGW #100163

Closed
MBCX opened this issue Dec 8, 2024 · 5 comments · Fixed by #99168
Closed

32-bit builds of Godot do not work with LLVM MinGW #100163

MBCX opened this issue Dec 8, 2024 · 5 comments · Fixed by #99168

Comments

@MBCX
Copy link
Contributor

MBCX commented Dec 8, 2024

Tested versions

  • Latest master (aa8d9b8)
  • all versions after 4.4dev5

System information

Godot v4.3.rc (322e93592) - Windows 10.0.26100 - GLES3 (Compatibility) - Radeon RX 580 Series (Advanced Micro Devices, Inc.; 31.0.21921.1000) - Intel(R) Core(TM) i5-9600K CPU @ 3.70GHz (6 Threads)

Issue description

Any attempt at building Godot for 32-bit Windows will succeed, but then actually trying to run will crash with this

================================================================
CrashHandlerException: Program crashed with signal 11
Engine version: Godot Engine v4.4.dev.custom_build (55e55b4879f7ad89086aa26f2e53882cf85077f9)
Dumping the backtrace.
-- END OF BACKTRACE --
================================================================

And just stop. And for some bizarre reason, it does not crash if its a dev build with debug symbols? (dev_mode="yes" dev_build="yes")
I'm still not sure if this is a Godot issue or a compiler issue. This is a Godot issue, see next comment.

Steps to reproduce

  1. Use the latest LLVM MinGW (can be older too, I tested 20231128
  2. Compile for windows with arch="x86_32"
  3. Wait for compilation and try to run the executable.
  4. (extra) re-compile with dev_mode="yes" dev_build="yes" and see how it does not crash.

Minimal reproduction project (MRP)

Use this link to get the builds and test it yourself:

@MBCX
Copy link
Contributor Author

MBCX commented Dec 10, 2024

Did a git bisect, I confirm this is a Godot bug, and the culprit is #85167.

@akien-mga
Copy link
Member

#99168 was merged yesterday and fixes some issues from #85167, could you check whether that fixes the 32-bit support too?

@MBCX
Copy link
Contributor Author

MBCX commented Dec 10, 2024

Built latest master (a372214) with LLVM 19.1.5, and it ran just fine!

RandomShaper thought ahead then 👌!

I'll do a production build (but with thin lto) to see if the issue is truly fixed.

@MBCX
Copy link
Contributor Author

MBCX commented Dec 10, 2024

Yes! it built and ran, and here's the final proof!

Image

So, I can close this issue then as a false alarm from my part.👍

@MBCX MBCX closed this as completed Dec 10, 2024
@akien-mga
Copy link
Member

Awesome, thanks for confirming the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Very Bad
Development

Successfully merging a pull request may close this issue.

3 participants