Re: PATCH: PR sanitizer/59136: llvm-symbolizer shouldn't be started always

2013-11-24 Thread Konstantin Serebryany
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 ab

Re: PATCH: PR sanitizer/59136: llvm-symbolizer shouldn't be started always

2013-11-24 Thread Jakub Jelinek
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/2

PATCH: PR sanitizer/59136: llvm-symbolizer shouldn't be started always

2013-11-24 Thread H.J. Lu
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? Thanks. H.J. 2013-11-24 H.J. Lu PR sanitizer/59136 * configure.ac: Add --enable-llvm-