]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Ensure USE_AVX... symbols are not defined if not building for x86_64.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 4 Jun 2026 14:37:25 +0000 (10:37 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 4 Jun 2026 14:37:25 +0000 (10:37 -0400)
commite88bd2736f86d66dc75ad8b2f7edd799d20b7d53
tree364145df2673dff9dc22acf7066ef1b0dc84c728
parentf0aedc7cb0ed82d4cca48332dc0852c767514558
Ensure USE_AVX... symbols are not defined if not building for x86_64.

Various code assumed this was true already, and usually it is.
However, it emerges that in a "universal" (multi-architecture)
macOS build, configure will define USE_AVX2_WITH_RUNTIME_CHECK
if the build host is x86_64, and then the arm64 half of the
build fails.

Ideally we'd get pg_config.h to define this symbol conditionally
depending on defined(__x86_64__), but I don't see any way to
persuade Autoconf to do that.  Instead, clean up the mess by
#undef'ing it again in c.h for not-x86_64 builds.

For consistency I made c.h also #undef the USE_AVX512... symbols.
Those are not actively broken, but it seems only happenstance
that configure's tests for them fail in a universal build.
Down the road we may have occasion to add more #undef's here.

This problem is new in v19, so no need for back-patch.

Reported-by: Sandeep Thakkar <sandeep.thakkar@enterprisedb.com>
Reported-by: Tobias Bussmann <t.bussmann@gmx.net>
Author: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/15574903-87C9-478A-B2D7-CC8F4C275DBB@gmx.net
src/include/c.h