This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFA:] Take 2 (was: Re: RFC: --enable-checking=valgrind.)
- From: Hans-Peter Nilsson <hp at bitrange dot com>
- To: Andreas Jaeger <aj at suse dot de>
- Cc: Richard Henderson <rth at redhat dot com>, <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 19 Nov 2002 12:07:21 -0500 (EST)
- Subject: Re: [RFA:] Take 2 (was: Re: RFC: --enable-checking=valgrind.)
On Tue, 19 Nov 2002, Andreas Jaeger wrote:
> Hans-Peter Nilsson <hp@bitrange.com> writes:
>
> > On Tue, 19 Nov 2002, Andreas Jaeger wrote:
> >
> >> With valgrind from CVS, there's one problem with your patch:
> >>
> >> /cvs/gcc/gcc/cppfiles.c: In function `read_include_file':
> >> /cvs/gcc/gcc/cppfiles.c:428: warning: implicit declaration of function `VALGRIND_DISCARD'
> >
> > Oh my, an API change. I use 1.0.4 as you might guess. I guess
>
> I used the latest CVS.
The snapshot I see is stated to be "of the current development,
unstable sources". Perhaps they will change this before
release, perhaps to something else.
(Where's the CVS repo? Perhaps it says in the 1.1.0 ss.)
> > that's easily autoconf:ed for, if you please.
>
> Can you do that as part of your patch - if not, I'll try to do it next
> weekend?
Please do it.
> Just checking for <memcheck.h> and including it if it exists should be
> all that's needed,
I think memcheck.h is just too common a name to do that without
risking breakage (I hope they change that, perhaps to
vgmemcheck.h). I suggest checking whether VALGRIND_DISCARD is
in valgrind.h, and if not, whether it's in memcheck.h and only
*then* define some macro to test before including memcheck.h.
I'll test with 1.0.4 if you do. But I can't really test that it
works for you with my current valgrind, so please do it.
(FWIW, there's no memcheck.h in 1.0.4.)
brgds, H-P