From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 107410 invoked by alias); 21 Dec 2016 10:40:02 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 107359 invoked by uid 89); 21 Dec 2016 10:40:01 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-5.0 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mx2.suse.de Received: from mx2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 21 Dec 2016 10:39:50 +0000 Received: from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 0194BAAE8; Wed, 21 Dec 2016 10:39:48 +0000 (UTC) User-Agent: K-9 Mail for Android In-Reply-To: <20161220185911.GM21933@tucnak> References: <20161220185911.GM21933@tucnak> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Subject: Re: [PATCH] Don't bootstrap libmpx unless needed From: Richard Biener Date: Wed, 21 Dec 2016 11:02:00 -0000 To: Jakub Jelinek CC: gcc-patches@gcc.gnu.org Message-ID: <61B998A1-13EF-4903-84EE-E00FA1A2C31B@suse.de> X-SW-Source: 2016-12/txt/msg01768.txt.bz2 On December 20, 2016 7:59:11 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >Similarly how we deal with bootstrapping libsanitizer only when >doing bootstrap-{a,u}san bootstrap, this avoids bootstrapping libmpx >if we don't need it for bootstrapping. > >Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? OK. Richard. >2016-12-20 Jakub Jelinek > > * configure.ac: Don't bootstrap libmpx unless --with-build-config > includes bootstrap-mpx. > * configure: Regenerated. > >--- configure.ac.jj1 2016-12-01 23:24:53.000000000 +0100 >+++ configure.ac 2016-12-20 10:50:08.715213438 +0100 >@@ -2643,9 +2643,14 @@ if echo " ${target_configdirs} " | grep > bootstrap_target_libs=${bootstrap_target_libs}target-libvtv, > fi > >-# If we are building libmpx, bootstrap it. >+# If we are building libmpx and $BUILD_CONFIG contains bootstrap-mpx, >+# bootstrap it. >if echo " ${target_configdirs} " | grep " libmpx " > /dev/null 2>&1; >then >- bootstrap_target_libs=${bootstrap_target_libs}target-libmpx, >+ case "$BUILD_CONFIG" in >+ *bootstrap-mpx* ) >+ bootstrap_target_libs=${bootstrap_target_libs}target-libmpx, >+ ;; >+ esac > fi > > # Determine whether gdb needs tk/tcl or not. >--- configure.jj1 2016-12-02 00:15:10.000000000 +0100 >+++ configure 2016-12-20 10:50:22.503034682 +0100 >@@ -7057,9 +7057,14 @@ if echo " ${target_configdirs} " | grep > bootstrap_target_libs=${bootstrap_target_libs}target-libvtv, > fi > >-# If we are building libmpx, bootstrap it. >+# If we are building libmpx and $BUILD_CONFIG contains bootstrap-mpx, >+# bootstrap it. >if echo " ${target_configdirs} " | grep " libmpx " > /dev/null 2>&1; >then >- bootstrap_target_libs=${bootstrap_target_libs}target-libmpx, >+ case "$BUILD_CONFIG" in >+ *bootstrap-mpx* ) >+ bootstrap_target_libs=${bootstrap_target_libs}target-libmpx, >+ ;; >+ esac > fi > > # Determine whether gdb needs tk/tcl or not. > > Jakub