[PATCH 0/1] MR6992: configure: Define _load_config_used symbol in the cross-compiler test program.
The load configuration is mandatory for ARM64EC binaries, and the MSVC linker emits a warning when it is missing. It makes sense, so I plan to add a similar warning to lld-link. Since the configure scripts treat linker warnings as errors, defining the load configuration ensures future compatibility. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6992
From: Jacek Caban <jacek(a)codeweavers.com> The load configuration is mandatory for ARM64EC binaries, and future versions of lld-link will issue a warning if it is missing. --- aclocal.m4 | 1 + 1 file changed, 1 insertion(+) diff --git a/aclocal.m4 b/aclocal.m4 index 9efbd7ebb3e..2ba926b490c 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -198,6 +198,7 @@ ac_wine_try_cflags_saved_exeext=$ac_exeext CFLAGS="$CFLAGS -nostdlib -nodefaultlibs $1" ac_exeext=".exe" AC_LINK_IFELSE([AC_LANG_SOURCE([[void *__os_arm64x_dispatch_ret = 0; +const unsigned int _load_config_used[0x50] = { sizeof(_load_config_used) }; #if defined(__clang_major__) && defined(MIN_CLANG_VERSION) && __clang_major__ < MIN_CLANG_VERSION #error Too old clang version #endif -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/6992
Hi, It looks like your patch introduced the new failures shown below. Please investigate and fix them before resubmitting your patch. If they are not new, fixing them anyway would help a lot. Otherwise please ask for the known failures list to be updated. The tests also ran into some preexisting test failures. If you know how to fix them that would be helpful. See the TestBot job for the details: The full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=150327 Your paranoid android. === debian11b (64 bit WoW report) === user32: input.c:4305: Test succeeded inside todo block: button_down_hwnd_todo 1: got MSG_TEST_WIN hwnd 00000000017600DE, msg WM_LBUTTONDOWN, wparam 0x1, lparam 0x320032
participants (3)
-
Jacek Caban -
Jacek Caban (@jacek) -
Marvin