dave/openttd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix stack alignment issues with 32 bit MinGW

Browse Source 
Set incoming and preferred alignment to 4 bytes to match Win32 standard

See: #704
This commit is contained in:
Jonathan G Rennison
2024-06-12 13:23:30 +01:00
parent c67f14dc88
commit c8ce89ff89
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
cmake/CompileFlags.cmake
View File

@@ -41,6 +41,11 @@ macro(compile_flags)
"$<$<CONFIG:Debug>:-Wa,-mbig-obj>" # Switch to pe-bigobj-x86-64 as x64 Debug builds push pe-x86-64 to the limits (linking errors with ASLR, ...) "$<$<CONFIG:Debug>:-Wa,-mbig-obj>" # Switch to pe-bigobj-x86-64 as x64 Debug builds push pe-x86-64 to the limits (linking errors with ASLR, ...)
) )
endif() endif()
if(CMAKE_SIZEOF_VOID_P EQUAL 4)
# Fix MinGW's incorrect assumption that the incoming stack at function calls is 16-byte aligned
# The Win32 API/calling convention only requires and guarantees 4-byte alignment, leading to alignment problems with SSE/AVX/etc
add_compile_options(-mincoming-stack-boundary=2 -mpreferred-stack-boundary=2)
endif()
add_compile_options(-Wno-stringop-overflow) # This warning false-positives on some MinGW versions so just turn it off add_compile_options(-Wno-stringop-overflow) # This warning false-positives on some MinGW versions so just turn it off
endif() endif()