Horton
Menu

Re: Centralised architecture detection

Tom Lane <tgl@sss.pgh.pa.us>

View thread
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: Greg Burd <greg@burd.me>, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Andres Freund <andres@anarazel.de>, Lukas Fittl <lukas@fittl.com>, John Naylor <johncnaylorls@gmail.com>, Bryan Green <dbryan.green@gmail.com>
Date: 2026-07-01T01:11:38Z
Lists: pgsql-hackers 
Thomas Munro <thomas.munro@gmail.com> writes:
> It looks like they didn't want __riscv32 and __riscv64?
> https://groups.google.com/a/groups.riscv.org/g/sw-dev/c/E8EO-Fd4t3s

Sigh ... another project that is convinced that they're smarter than
everybody else and conforming to common practice is an anti-pattern.

Based on that thread, I'm thinking

...
#elif defined(__riscv)
#if SIZEOF_VOID_P == 8
#define __riscv64__ 1
#else
#define __riscv__ 1
#endif
#elif defined(__s390__)
...

I'd rather rely on our own pointer-size determination than YA
magic compiler-defined symbol.

			regards, tom lane

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Fix CPU-identification macros for RISC-V.

  2. Clean up inconsistencies in CPU-identification macros.