This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 12/21] PR jit/63854: Add a valgrind suppresion file

From: Jeff Law <law at redhat dot com>
To: Richard Biener <richard dot guenther at gmail dot com>, David Malcolm <dmalcolm at redhat dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, jit at gcc dot gnu dot org
Date: Wed, 19 Nov 2014 10:09:34 -0700
Subject: Re: [PATCH 12/21] PR jit/63854: Add a valgrind suppresion file
Authentication-results: sourceware.org; auth=none
References: <1416393981-39626-1-git-send-email-dmalcolm at redhat dot com> <1416393981-39626-13-git-send-email-dmalcolm at redhat dot com> <CAFiYyc1BV4k5Uc8v-xyivYnqngXgP+sxP3uN7R87iMn1hTxB2Q at mail dot gmail dot com>

On 11/19/14 04:47, Richard Biener wrote:

On Wed, Nov 19, 2014 at 11:46 AM, David Malcolm <dmalcolm@redhat.com> wrote:

Valgrind complains about uninitialized data within sparseset_bit_p.
Provide a suppression file to silence these warnings.

Valgrind requires suppression files for C++ code to use the mangled
names, so we do that here.


There is --enable-valgrind-annotations to get the same effect by GCC
telling valgrind about this (and more).

Right.  See VALGRIND_DISCARD.  Is that not covering this case?


Jeff

Follow-Ups:
- Re: [PATCH 12/21] PR jit/63854: Add a valgrind suppresion file
  - From: David Malcolm

References:
- [PATCH 00/21] PR 63854: Fix various memory leaks
  - From: David Malcolm
- [PATCH 12/21] PR jit/63854: Add a valgrind suppresion file
  - From: David Malcolm
- Re: [PATCH 12/21] PR jit/63854: Add a valgrind suppresion file
  - From: Richard Biener

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]