This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: PR sanitizer/59136: llvm-symbolizer shouldn't be started always
- From: Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Dodji Seketeli <dodji at redhat dot com>, Kostya Serebryany <kcc at google dot com>
- Date: Sun, 24 Nov 2013 22:47:39 +0400
- Subject: Re: PATCH: PR sanitizer/59136: llvm-symbolizer shouldn't be started always
- Authentication-results: sourceware.org; auth=none
- References: <20131124182018 dot GA23926 at intel dot com> <20131124184232 dot GQ892 at tucnak dot redhat dot com>
Besides all changes to .cc/.h files in libsanitizer should go thorough
the upstream repo first
and I generally dislike the change in
libsanitizer/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc,
we'll need to find another solution (using libbacktrace could be one;
I am not so enthusiastic about it being easy,
but will be glad to be wrong).
--kcc
On Sun, Nov 24, 2013 at 10:42 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> On Sun, Nov 24, 2013 at 10:20:18AM -0800, H.J. Lu wrote:
>> Hi,
>>
>> This patch adds --enable-llvm-symbolizer and disables llvm-symbolizer by
>> default. Without it, GCC sanitizer is unusable on systems with
>> unsupported /bin/llvm-symbolizer. OK to install?
>
> See http://gcc.gnu.org/ml/gcc-patches/2013-11/msg02055.html
> and following thread, I hope in a week or so we will just use libbacktrace
> in GCC for symbolization.
>
> Jakub