Re: Broken ./configure checks for __cpuid() and __cpuidex()
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Lukas Fittl <lukas@fittl.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-07-30T08:13:50Z
Lists: pgsql-hackers
On Wed, Jul 30, 2025 at 12:02:44PM +0900, Michael Paquier wrote: > On Tue, Jul 29, 2025 at 12:21:32AM -0700, Lukas Fittl wrote: >> FWIW, I looked again at the MinGW sources and I think you're right that >> intrin.h is likely also correct for MinGW. I originally thought that >> cpuid.h would be correct, given that's whats used by GCC/clang, but I think >> the MinGW source itself is authoritative here (vs the compiler in use), and >> that has a intrin.h include ([0]) but no cpuid.h. > > Yeah. I have a close of MinGW locally, and that matches what I saw. s/close/clone/. > Applied on all the branches. Let's wait for the buildfarm to report, > now. The first reports are popping up for the Windows animals, and as far as I can see things seem to tbe OK for now in terms of the flavors of __cpuid & friends detected, for the mix of meson, MinGW, MSVC and configure that we have. -- Michael
Commits
-
Fix ./configure checks with __cpuidex() and __cpuid()
- 612f5b80620d 13.22 landed
- 60953d4cba7c 14.19 landed
- d6ffc43f95b2 15.14 landed
- c1984be23cdc 16.10 landed
- 8de56323c139 17.6 landed
- cd2d52cc6b39 18.0 landed
- 1a5212775e46 19 (unreleased) landed