MIME-Version: 1.0
References: 
 <CA+vA85_5GTu+HHniSbvvP+8k3=xZO=WE84NPwiKyxztqvpfZ3Q@mail.gmail.com>
 <CA+vA8587=RpZuL+EJ=XKM-8-xdPxz2mFexbAjaimN+OkXysXqw@mail.gmail.com>
 <CA+Yyo5RihTRUdUdanuNYhjQeXQY6412FWjzaxJAQR5MGX83=EQ@mail.gmail.com>
 <CA+Yyo5Qw=v2bcPSeyGgX3WuHeDXm7vxPcET1n+yUreUmO0rk_A@mail.gmail.com>
 <CA+vA858b4J1k_34vRR+=UsM5U4JSVc4QkigHExk0RYH9nk3iRQ@mail.gmail.com>
 <CAN55FZ3pWRo_uettqKdAAtYyYSgrc9rnVY3H0Mfs-PHYuhPaKw@mail.gmail.com>
 <CA+vA85_Er75cm7-0LWdiBbQvzk57xkOXH3jQ7bZTQiHGzjMC9Q@mail.gmail.com>
In-Reply-To: 
 <CA+vA85_Er75cm7-0LWdiBbQvzk57xkOXH3jQ7bZTQiHGzjMC9Q@mail.gmail.com>
From: John Naylor <johncnaylorls@gmail.com>
Date: Mon, 2 Jun 2025 19:11:45 +0700
Message-ID: 
 <CANWCAZb=CDRMMERC7FA=-0mN4hQkToSsPHgBnAV3X1bz=8ONuQ@mail.gmail.com>
Subject: Re: Proposal for enabling auto-vectorization for checksum
 calculations
To: Matthew Sterrett <matthewsterrett2@gmail.com>
Cc: Nazir Bilal Yavuz <byavuz81@gmail.com>, Stepan Neretin <slpmcf@gmail.com>,
	pgsql-hackers@lists.postgresql.org,
	"Devulapalli, Raghuveer" <raghuveer.devulapalli@intel.com>,
	"Shankaran, Akash" <akash.shankaran@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CANWCAZb%3DCDRMMERC7FA%3D-0mN4hQkToSsPHgBnAV3X1bz%3D8ONuQ%40mail.gmail.com>
Precedence: bulk

On Fri, May 23, 2025 at 4:54=E2=80=AFAM Matthew Sterrett
<matthewsterrett2@gmail.com> wrote:
> Okay, I've determined that versions of LLVM/Clang before 19 crash when
> compiling this patch for some reason; it seems that both make
> check-world and make install will crash with the affected LLVM
> versions.
> Unfortunately, what matters seems to be the version of the linker/LTO
> optimizer, which I don't think we can check at compile time.
> I added a check for Clang>=3D19 which works at preventing the crash on my=
 system.
> I think it's possible some unusual combination of clang/LLVM might
> still crash during the build, but I think this is a reasonable
> solution

I don't know if this is related to the crashes, but it doesn't seem
like a good idea to #include the function pointer stuff everywhere,
that should probably go into src/port like the others.

--
John Naylor
Amazon Web Services