From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com> |
Cc: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk>, Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: wrong query result with jit_above_cost= 0 |
Date: | 2018-07-23 04:24:27 |
Message-ID: | 20180723042427.54rxm7qyats73tqh@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 2018-07-22 17:09:37 -0700, Andres Freund wrote:
> > Also, I found one suspicious thing, in AGG_PLAIN_TRANS section we don't
> > switch the memory context back in the branch with ExecAggTransReparent. I
> > never found any consequences of that, but just in case I believe it makes sense
> > to do so.
>
> I'll look at that next.
That's indeed necessary. Pushed. Thanks!
>
> > And the last thing - where can I find a documentation about how to properly
> > apply patches for GDB & perf support to llvm? I remember they were posted here,
> > and found some of them here [1] from Andres, but apparently part of them was
> > already applied on top of llvm. Looks like for the gdb support I need to apply
> > 0006-ORC-JIT-event-listener-support (since there is a gdb listener mentioned
> > there), but with this patch I have an error:
> >
> > error: ‘ObjHandleT’ was not declared in this scope
> >
> > So I'm confused how should it be?
>
> I've merged the GDB part into LLVM, and am about to merge the perf part
> too. I plan to push a fix to PG adapting it to use the agreed upon /
> merged LLVM APIs. Then you'll just need a recent LLVM
> checkout. Unfortunately the relevant LLVM internal APIs have changed
> quite rapidly over the last few releases (and a lot within individual
> releases), so it's not easy to provide a patch for the individual versions.
Adapted the PG bits.
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2018-07-23 04:39:40 | Re: Should contrib modules install .h files? |
Previous Message | Tom Lane | 2018-07-23 04:15:53 | Re: Should contrib modules install .h files? |