diff options
| author | Tom Tromey <tromey@adacore.com> | 2023-01-09 07:43:29 -0700 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2023-01-12 07:20:46 -0700 |
| commit | 3cc048fc6b1f29ea7f5de83815e56884bbd05f61 (patch) | |
| tree | 486a3974641a32b27cda9269da37769fb2a0a3c1 /gdbsupport | |
| parent | 86ac3f1401297070ede1f1d3aac247260c14cd84 (diff) | |
| download | gdb-3cc048fc6b1f29ea7f5de83815e56884bbd05f61.zip gdb-3cc048fc6b1f29ea7f5de83815e56884bbd05f61.tar.gz gdb-3cc048fc6b1f29ea7f5de83815e56884bbd05f61.tar.bz2 | |
Set _WIN32_WINNT in common.m4 configure check
GCC recently added support for the Windows thread model, enabling
libstdc++ to support Windows natively. However, this supporrt
requires a version of Windows later than the minimum version that is
supported by GDB.
PR build/29966 points out that the GDB configure test for std::thread
does not work in this situation, because _WIN32_WINNT is not defined
in test program, and so <thread> seems to be fine.
This patch is an attempt to fix the problem, by using the same setting
for _WIN32_WINNT at configure time as is used at build time.
I don't have access to one of the older systems so I don't think I can
truly test this. I did do a mingw cross build, though. I'm going to
ask the bug reporter to test it.
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29966
(cherry picked from commit 2124b2de4bba05063218501dcccfc4cffe648cda)
Diffstat (limited to 'gdbsupport')
| -rw-r--r-- | gdbsupport/common-defs.h | 4 | ||||
| -rw-r--r-- | gdbsupport/common.m4 | 13 | ||||
| -rwxr-xr-x | gdbsupport/configure | 12 |
3 files changed, 26 insertions, 3 deletions
diff --git a/gdbsupport/common-defs.h b/gdbsupport/common-defs.h index bcebd71..5e529f6 100644 --- a/gdbsupport/common-defs.h +++ b/gdbsupport/common-defs.h @@ -70,7 +70,9 @@ /* We don't support Windows versions before XP, so we define _WIN32_WINNT correspondingly to ensure the Windows API headers - expose the required symbols. */ + expose the required symbols. + + NOTE: this must be kept in sync with common.m4. */ #if defined (__MINGW32__) || defined (__CYGWIN__) # ifdef _WIN32_WINNT # if _WIN32_WINNT < 0x0501 diff --git a/gdbsupport/common.m4 b/gdbsupport/common.m4 index 7b10026..b3aa271 100644 --- a/gdbsupport/common.m4 +++ b/gdbsupport/common.m4 @@ -100,7 +100,18 @@ AC_DEFUN([GDB_AC_COMMON], [ AC_CACHE_CHECK([for std::thread], gdb_cv_cxx_std_thread, [AC_COMPILE_IFELSE([AC_LANG_PROGRAM( - [[#include <thread> + dnl NOTE: this must be kept in sync with common-defs.h. + [[#if defined (__MINGW32__) || defined (__CYGWIN__) + # ifdef _WIN32_WINNT + # if _WIN32_WINNT < 0x0501 + # undef _WIN32_WINNT + # define _WIN32_WINNT 0x0501 + # endif + # else + # define _WIN32_WINNT 0x0501 + # endif + #endif /* __MINGW32__ || __CYGWIN__ */ + #include <thread> void callback() { }]], [[std::thread t(callback);]])], gdb_cv_cxx_std_thread=yes, diff --git a/gdbsupport/configure b/gdbsupport/configure index 38541e8..44dd8a9 100755 --- a/gdbsupport/configure +++ b/gdbsupport/configure @@ -8957,7 +8957,17 @@ if ${gdb_cv_cxx_std_thread+:} false; then : else cat confdefs.h - <<_ACEOF >conftest.$ac_ext /* end confdefs.h. */ -#include <thread> + #if defined (__MINGW32__) || defined (__CYGWIN__) + # ifdef _WIN32_WINNT + # if _WIN32_WINNT < 0x0501 + # undef _WIN32_WINNT + # define _WIN32_WINNT 0x0501 + # endif + # else + # define _WIN32_WINNT 0x0501 + # endif + #endif /* __MINGW32__ || __CYGWIN__ */ + #include <thread> void callback() { } int main () |