From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7383 invoked by alias); 5 Nov 2008 18:32:45 -0000 Received: (qmail 7300 invoked by uid 22791); 5 Nov 2008 18:32:30 -0000 X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.33.17) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 05 Nov 2008 18:31:35 +0000 Received: from spaceape14.eur.corp.google.com (spaceape14.eur.corp.google.com [172.28.16.148]) by smtp-out.google.com with ESMTP id mA5IVVmi003423; Wed, 5 Nov 2008 18:31:31 GMT Received: from localhost.localdomain.google.com (dhcp-172-22-125-203.corp.google.com [172.22.125.203]) (authenticated bits=0) by spaceape14.eur.corp.google.com with ESMTP id mA5IVRaM012659 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Wed, 5 Nov 2008 10:31:29 -0800 To: Rainer Orth Cc: gcc-patches@gcc.gnu.org Subject: Re: 4.3/4.4 PATCH: Fix Solaris/x86 bootstrap with Sun ld (PR bootstrap/33100) References: <18671.26770.804364.445962@manam.TechFak.Uni-Bielefeld.DE> <18705.54181.858026.33670@manam.TechFak.Uni-Bielefeld.DE> From: Ian Lance Taylor Date: Wed, 05 Nov 2008 18:32:00 -0000 In-Reply-To: <18705.54181.858026.33670@manam.TechFak.Uni-Bielefeld.DE> (Rainer Orth's message of "Wed\, 5 Nov 2008 18\:11\:01 +0100 \(MET\)") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-IsSubscribed: yes 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 X-SW-Source: 2008-11/txt/msg00183.txt.bz2 Rainer Orth writes: > As I suggested in the original patch submission, it should be possible to > move the inclusion of i386/t-crtstuff to libgcc/configure.ac and make it > conditional on a linker test there. This way, only systems with broken Sun > linkers would be penalized. I could well try that approach, since it's > certainly cleaner and doesn't break unwind information on recent Solaris 11 > where we don't have to. What do you think? I think it's up to you. You can also just split the config.gcc test on the solaris version number. >> I'll approve this patch for mainline and 4.3 branch. > > What about the 4.2 branch? As I mentioned, the patch is needed there, too, > and I've successfully bootstrapped on i386-pc-solaris2.10. Unfortunately, > the cleaner approach outlined above isn't available here since there's no > separate libgcc with it's own configure, so I fear it's either this hack or > leave that branch broken. OK for 4.2 branch also. Ian