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

  1. Fix ./configure checks with __cpuidex() and __cpuid()