This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
Re: [PATCH 1/2] Add configure options for Valgrind annotations.
- From: Mark Wielaard <mark at klomp dot org>
- To: elfutils-devel at sourceware dot org
- Cc: Jonathon Anderson <jma14 at rice dot edu>
- Date: Thu, 07 Nov 2019 18:19:26 +0100
- Subject: Re: [PATCH 1/2] Add configure options for Valgrind annotations.
- References: <1572380520.19948.0@rice.edu> <20191029211437.3268-1-mark@klomp.org>
Hi Jonathon,
On Tue, 2019-10-29 at 22:14 +0100, Mark Wielaard wrote:
> +2019-08-25 Jonathon Anderson <jma14@rice.edu>
> +
> + * configure.ac: Add new --enable-valgrind-annotations
> + * configure.ac: Add new --with-valgrind (headers only)
I think this looks fine. In theory we could also try to use pkg-config
checks, since valgrind does ship a valgrind.pc file. But various
distros don't actually ship that :{ So it is probably not worth it.
I must admit that I don't actually understand how the annotations
really work, see comments on the other patch.
Thanks,
Mark