From patchwork Thu Nov 15 16:28:55 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: PR other/55291: Add support for GCC multilib run-time libraries to libsanitizer From: "H.J. Lu" X-Patchwork-Id: 199342 Message-Id: To: Paolo Bonzini Cc: Markus Trippelsdorf , Jakub Jelinek , Alexandre Oliva , Dodji Seketeli , DJ Delorie , Ralf Wildenhues , gcc-patches@gcc.gnu.org Date: Thu, 15 Nov 2012 08:28:55 -0800 On Thu, Nov 15, 2012 at 8:08 AM, Paolo Bonzini wrote: > Il 15/11/2012 17:02, H.J. Lu ha scritto: >>>> >> I can reproduce it with --enable-version-specific-runtime-libs. I am >>>> >> taking a look. >>>> >> >>> > >>> > I am checking in this patch. >>> > >> This is what I checked in. > > libssp et al. do not need any of this stuff. > > What's special about libsanitizer? > > Paolo Did you mean the it since they build libraries in subdirectories. From libstdc++-v3/configure.ac: # Multilibs need MULTISUBDIR defined correctly in certain makefiles so # that multilib installs will end up installed in the correct place. # The testsuite needs it for multilib-aware ABI baseline files. # To work around this not being passed down from config-ml.in -> # srcdir/Makefile.am -> srcdir/{src,libsupc++,...}/Makefile.am, manually # append it here. Only modify Makefiles that have just been created. # diff --git a/libsanitizer/configure.ac b/libsanitizer/configure.ac index 27a7f95..76a19b6 100644 --- a/libsanitizer/configure.ac +++ b/libsanitizer/configure.ac @@ -82,6 +82,9 @@ _EOF mv tmp$$ $ac_file rm vpsed$$ echo 'MULTISUBDIR =' >> $ac_file + ml_norecursion=yes + . ${multi_basedir}/config-ml.in + AS_UNSET([ml_norecursion]) ]) part? libstdc++ and libsanitizer are only libraries which need