Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-12-30 19:14:31 +00:00
Code Issues Projects Releases Packages Wiki Activity

[CMAKE] Remove "/Gy" option from ML

Browse source 
Reason: VS solution builds pass the compiler options to the assembler. These are usually ignored and result in a warning only, but ML of the latest VS supports /Gy (function level linking), but that requires all functions to be declared with "PROC" and otherwise results in an error.
This commit is contained in:
Timo Kreuzer 2023-11-11 21:38:59 +02:00
parent 72081168fb
commit e3f204e228
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
sdk/cmake/msvc.cmake
View file

@ -22,8 +22,9 @@ endif()
# helper macros. Note also that GCC builds use string pooling by default. # helper macros. Note also that GCC builds use string pooling by default.
add_compile_options(/GF) add_compile_options(/GF)
# Enable function level linking and comdat folding # Enable function level linking and comdat folding (only C/C++, not ASM!)
add_compile_options(/Gy) add_compile_options($<$<COMPILE_LANGUAGE:CXX>:/Gy>)
add_compile_options($<$<COMPILE_LANGUAGE:C>:/Gy>)
add_link_options(/OPT:REF /OPT:ICF) add_link_options(/OPT:REF /OPT:ICF)
if(ARCH STREQUAL "i386") if(ARCH STREQUAL "i386")