MIME-Version: 1.0
References: <aYN5vmHb89m3sfgs@nathan> <1472307.1770229663@sss.pgh.pa.us>
 <aYOTFLqyvvPUQLN8@nathan> <ae7BvFTamKBDF1ga@nathan>
 <2446899.1777256313@sss.pgh.pa.us>
 <ae7KiQcfDaavu5cD@nathan> <ae-8WBrqoTWYCTCG@nathan>
 <CANWCAZYFAP_fP0RSnor9Aa03aYAvQaM=+8oNeZSV432RNNynkQ@mail.gmail.com>
 <2871418.1777382981@sss.pgh.pa.us> <afC_haf7bXWHb6e9@nathan>
 <afEZXCDzmlr3TnSg@nathan>
In-Reply-To: <afEZXCDzmlr3TnSg@nathan>
From: John Naylor <johncnaylorls@gmail.com>
Date: Wed, 29 Apr 2026 12:04:48 +0700
Message-ID: 
 <CANWCAZZVVhymm2eqzxQczi9pX7C6sKDzdsOT89v9CtZB__+p+g@mail.gmail.com>
Subject: 
 =?UTF-8?Q?Re=3A_=5BBUG=3F=5D_macOS_=28Intel=29_build_warnings=3A_=22ranlib=3A_fi?=
	=?UTF-8?Q?le_=E2=80=A6_has_no_symbols=22_for_aarch64_objects?=
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Peter Eisentraut <peter@eisentraut.org>,
	Zhang Mingli <zmlpostgres@gmail.com>,
	PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CANWCAZZVVhymm2eqzxQczi9pX7C6sKDzdsOT89v9CtZB__%2Bp%2Bg%40mail.gmail.com>
Precedence: bulk

On Wed, Apr 29, 2026 at 3:32=E2=80=AFAM Nathan Bossart <nathandbossart@gmai=
l.com> wrote:
>
> On Tue, Apr 28, 2026 at 09:09:09AM -0500, Nathan Bossart wrote:
> > On Tue, Apr 28, 2026 at 09:29:41AM -0400, Tom Lane wrote:
> >> If we do care, a low-tech solution could be to have a dummy symbol
> >> declared in each file, along the lines of
> >>
> >> #if HAVE_X86_64_POPCNTQ
> >> // code for popcntq case
> >> #elif HAVE_FROBNITZ
> >> // code for frobnitz case
> >> ...
> >> #else
> >> /* prevent linker complaints about empty module */
> >> extern int pg_cpu_x86_dummy_variable;
> >> int  pg_cpu_x86_dummy_variable =3D 0;
> >> #endif
> >>
> >> Actually then we'd not need to change the build systems either...
> >
> > I've been trying to avoid doing that, but it's a far simpler solution, =
and
> > in theory it should fix the problem for all platforms, too.  So, it's
> > probably the way to go.
>
> Here's a full set of patches for back-patching purposes.

+1

-#endif           /* USE_NEON */
+#else
...
+#endif

Is the removal of the comment intentional?

--=20
John Naylor
Amazon Web Services