Compilation failure with `lto=full` on Windows with MSVC `cl.exe`: LNK1248: image size exceeds maximum allowable size (FFFFFFFF)

[Swarkin]

Tested versions

master f032af7

System information

Windows 11 - Intel i5 9600k

Issue description

Compilation randomy fails after a while:

[...}
Compiling scene\gui\box_container.cpp ...
Compiling scene\gui\button.cpp ...
Compiling scene\gui\center_container.cpp ...
Compiling scene\gui\check_box.cpp ...
editor\editor.windows.editor.x86_64.lib : fatal error LNK1248: image size (100764120) exceeds maximum allowable size (FFFFFFFF)
Compiling scene\gui\check_button.cpp ...
Compiling scene\gui\code_edit.cpp ...
Compiling scene\gui\color_mode.cpp ...
Compiling scene\gui\color_picker.cpp ...
Compiling scene\gui\color_rect.cpp ...
Compiling scene\gui\container.cpp ...
scons: *** [editor\editor.windows.editor.x86_64.lib] Error 1248
scons: building terminated because of errors.
[Time elapsed: 00:08:12.40]

Steps to reproduce

clone godot
scons p=windows arch=x86_64 production=yes lto=full target=editor optimize=speed deprecated=no

Minimal reproduction project (MRP)

[matheusmdx]

Bisecting points to #93311 as the culprit, @dsnopek

[smix8]

It is the lto=full causing the Error 1248.
Builds fine with other build options that dont use lto.

[Swarkin]

Still relevant

[hpvb]

Does the problem occur when using clang-cl.exe instead of cl.exe? It can be installed through the visual studio install tool

[dsnopek]

Sorry for not taking a look at this sooner!

I am able to reproduce locally with scons p=windows lto=full on current master.

However, even if I try completely reverting PR #93311 (so, current master but with those changes reverted, plus removing some other new usages of GDVIRTUAL*_REQUIRED()), I still get the same LNK1248 error.

So, I think my PR was more "the straw that broke the camels back", rather a problematic change, since it seems that other changes after my PR will cause the same issue.

[YYF233333]

This is a limitation of COFF format. See https://developercommunity.visualstudio.com/t/static-lib-4gb-file-size-limit-lnk1248-image-size/332991

COFF has a 4GB size limit - offset fields in COFF won’t be able to encode anything larger than 4GB due to fields being 4-byte wide.

The only way to workaround this seems to be split the lib file.

Strange though without LTO this lib is only 239MB.

Checking other lib files, LTO bloat file size by a factor of 5-10x.
text_server_adv: 10MB -> 46MB
webrtc: 2MB -> 23MB

clang-cl doesn't have this issue, use_llvm=yes use_lto=full produce libs even smaller than msvc without lto.