autoconf: Unify CFLAGS_SSE42 and CFLAGS_ARMV8_CRC32C
authorAndres Freund <andres@anarazel.de>
Fri, 2 Dec 2022 02:46:55 +0000 (18:46 -0800)
committerAndres Freund <andres@anarazel.de>
Fri, 2 Dec 2022 02:46:55 +0000 (18:46 -0800)
commite0f0e08e17a6186ce299ed8a4385a7a486f304ed
tree2fa17957a3766ff2deadbeeabd1fbe5b724d94d2
parent5e73a6048849bd7bda4947e39570b9011734114d
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
config/c-compiler.m4
configure
configure.ac
src/Makefile.global.in
src/port/Makefile