Re: Requiring LLVM 14+ in PostgreSQL 18

Peter Eisentraut <peter@eisentraut.org>

From: Peter Eisentraut <peter@eisentraut.org>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2024-05-15T05:20:09Z
Lists: pgsql-hackers
On 15.05.24 06:21, Thomas Munro wrote:
>> And as I'm looking up how this was previously handled, I notice that
>> this list of clang-NN versions was last updated equally sneakily as part
>> of your patch to trim off LLVM <10 (820b5af73dc).  I wonder if the
>> original intention of that configure code was that maintaining the
>> versioned list above clang-7/llvm-config-7 was not needed, because the
>> unversioning programs could be used, or maybe because pkg-config could
>> be used.  It would be nice if we could get rid of having to update that.
> I probably misunderstood why we were doing that, perhaps something to
> do with the way some distro (Debian?) was doing things with older
> versions, and yeah I see that we went a long time after 7 without
> touching it and nobody cared.  Yeah, it would be nice to get rid of
> it.  Here's a patch.  Meson didn't have that.

Yes, let's get that v3-0001 patch into PG17.




Commits

  1. doc: remove llvm-config search from configure documentation

  2. jit: Remove {llvm-config,clang}-N configure probes.