pgsql: autoconf: Unify CFLAGS_SSE42 and CFLAGS_ARMV8_CRC32C - Mailing list pgsql-committers

From Andres Freund
Subject pgsql: autoconf: Unify CFLAGS_SSE42 and CFLAGS_ARMV8_CRC32C
Date
Msg-id E1p0wIk-001e7y-PT@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
autoconf: Unify CFLAGS_SSE42 and CFLAGS_ARMV8_CRC32C

Until now we emitted the cflags to build the CRC objects into architecture
specific variables. That doesn't make a whole lot of sense to me - we're never
going to target x86 and arm at the same time, so they don't need to be
separate variables.

It might be better to instead continue to have CFLAGS_SSE42 /
CFLAGS_ARMV8_CRC32C be computed by PGAC_ARMV8_CRC32C_INTRINSICS /
PGAC_SSE42_CRC32_INTRINSICS and then set CFLAGS_CRC based on those. But it
seems unlikely that we'd need other sets of CRC specific flags for those two
architectures at the same time.

This simplifies the upcoming meson PGXS compatibility.

Reviewed-by: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Discussion: https://postgr.es/m/20221005200710.luvw5evhwf6clig6@awork3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/e0f0e08e17a6186ce299ed8a4385a7a486f304ed

Modified Files
--------------
config/c-compiler.m4   |  8 ++++----
configure              | 19 +++++++++----------
configure.ac           | 10 +++++-----
src/Makefile.global.in |  3 +--
src/port/Makefile      | 18 +++++++++---------
5 files changed, 28 insertions(+), 30 deletions(-)


pgsql-committers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: pgsql: Expand AclMode to 64 bits
Next
From: Andres Freund
Date:
Subject: pgsql: autoconf: Don't AC_SUBST() LD in configure