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
-
doc: remove llvm-config search from configure documentation
- 0c7ec3b3a03b 17.0 landed
- 4baff5013277 18.0 landed
-
jit: Remove {llvm-config,clang}-N configure probes.
- 4dd29b683334 17.0 landed