From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Pg Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: coverage additions |
Date: | 2019-05-30 19:28:05 |
Message-ID: | 20190530192805.GA25189@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2019-May-30, Tom Lane wrote:
> Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
> > I just enabled --enabled-llvm on the coverage reporting machine, which
> > made src/backend/jit/jit.c go from 60/71 % (line/function wise) to 78/85 % ...
> > and src/backend/jit/llvm from not appearing at all in the report to
> > 78/94 %. That's a good improvement.
>
> > If there are other obvious improvements to be had, please let me know.
>
> I was going to suggest that adding some or all of
>
> -DCOPY_PARSE_PLAN_TREES
> -DWRITE_READ_PARSE_PLAN_TREES
> -DRAW_EXPRESSION_COVERAGE_TEST
>
> to your CPPFLAGS might improve the reported coverage in backend/nodes/,
> and perhaps other places.
I did that, and it certainly increased backend/nodes numbers
considerably. Thanks.
(extensible.c remains at 0% though, as does its companion nodeCustom.c).
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2019-05-30 20:23:11 | Re: coverage additions |
Previous Message | Larry Rosenman | 2019-05-30 18:39:30 | Re: New committer: David Rowley |