Re: pgsql: Track LLVM 15 changes.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Thomas Munro <tmunro(at)postgresql(dot)org>, pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org>
Subject: Re: pgsql: Track LLVM 15 changes.
Date: 2025-12-25 17:05:19
Message-ID: 1407185.1766682319@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

[ blast-from-the-past dept. ]

Thomas Munro <thomas(dot)munro(at)gmail(dot)com> writes:
> Shortly I'll do the back-patch of LLVM 14 API changes (their final
> release tag is scheduled for tomorrow, as I've been reminded by
> package maintainers trying and failing to build). But I figured I
> should also follow up on this bandaid solution to the API deprecation
> warnings, that is, silencing them for now, so we don't spew screeds of
> warnings. Does someone have a better idea than the attached?

The bit added by commit a56e7b660 is still there in
src/backend/jit/llvm/Makefile:

+# LLVM 14 produces deprecation warnings. We'll need to make some changes
+# before the relevant functions are removed, but for now silence the warnings.
+ifeq ($(GCC), yes)
+LLVM_CFLAGS += -Wno-deprecated-declarations
+endif

Surely we don't need this anymore? I tried removing it locally
and didn't see any such warnings (with Red Hat's llvm 20.1.8).

regards, tom lane

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2025-12-25 22:54:59 pgsql: Fix typo in stat_utils.c
Previous Message Michael Paquier 2025-12-25 06:17:02 pgsql: Move attribute statistics functions to stat_utils.c