From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
Cc: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: broken JIT support on Fedora 40 |
Date: | 2024-04-10 18:54:46 |
Message-ID: | CAFj8pRAfnqdt=efcR8-1Zw8txnt6Wa5F=7RcBX4BdqtYLyBH=g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
st 10. 4. 2024 v 2:44 odesílatel Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
napsal:
> On Tue, Apr 9, 2024 at 10:05 PM Dmitry Dolgov <9erthalion6(at)gmail(dot)com>
> wrote:
> > + /* In assertion builds, run the LLVM verify pass. */
> > +#ifdef USE_ASSERT_CHECKING
> > + LLVMPassBuilderOptionsSetVerifyEach(options, true);
> > +#endif
>
> Thanks, that seems nicer. I think the question is whether it will
> slow down build farm/CI/local meson test runs to a degree that exceeds
> its value. Another option would be to have some other opt-in macro,
> like the existing #ifdef LLVM_PASS_DEBUG, for people who maintain
> JIT-related stuff to turn on.
>
> Supposing we go with USE_ASSERT_CHECKING, I have another question:
>
> - const char *nm = "llvm.lifetime.end.p0i8";
> + const char *nm = "llvm.lifetime.end.p0";
>
> Was that a mistake, or did the mangling rules change in some version?
> I don't currently feel inclined to go and test this on the ancient
> versions we claim to support in back-branches. Perhaps we should just
> do this in master, and then it'd be limited to worrying about LLVM
> versions 10-18 (see 820b5af7), which have the distinct advantage of
> being available in package repositories for testing. Or I suppose we
> could back-patch, but only do it if LLVM_VERSION_MAJOR >= 10. Or we
> could do it unconditionally, and wait for ancient-LLVM build farm
> animals to break if they're going to.
>
> I pushed the illegal attribute fix though. Thanks for the detective work!
>
> (It crossed my mind that perhaps deform functions should have their
> own template function, but if someone figures out that that's a good
> idea, I think we'll *still* need that change just pushed.)
>
all tests passed on fc 40 without problems
Thank you
Pavel
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2024-04-10 19:08:21 | Re: Issue with the PRNG used by Postgres |
Previous Message | Kirill Reshke | 2024-04-10 18:39:48 | Re: psql: Greatly speed up "\d tablename" when not using regexes |