From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19532 invoked by alias); 24 Jul 2006 13:55:59 -0000 Received: (qmail 19524 invoked by uid 22791); 24 Jul 2006 13:55:59 -0000 X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (66.187.233.31) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 24 Jul 2006 13:55:54 +0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.12.11.20060308/8.12.11) with ESMTP id k6ODtq96016529; Mon, 24 Jul 2006 09:55:52 -0400 Received: from pobox.surrey.redhat.com (pobox.surrey.redhat.com [172.16.10.17]) by int-mx1.corp.redhat.com (8.12.11.20060308/8.12.11) with ESMTP id k6ODtpF7003192; Mon, 24 Jul 2006 09:55:52 -0400 Received: from [10.32.68.6] (vpn-68-6.surrey.redhat.com [10.32.68.6]) by pobox.surrey.redhat.com (8.12.11.20060308/8.12.11) with ESMTP id k6ODtoTR020980; Mon, 24 Jul 2006 14:55:50 +0100 Message-ID: <44C4D165.7000900@redhat.com> Date: Mon, 24 Jul 2006 13:55:00 -0000 From: Nick Clifton User-Agent: Thunderbird 1.5.0.4 (X11/20060516) MIME-Version: 1.0 To: Andrew STUBBS CC: binutils@sourceware.org Subject: Re: [PATCH]: Set little endian flag when linking SH arch tests for little endian multilibs References: <44C0D59D.70306@st.com> <44C386B9.2050504@redhat.com> <44C48481.8090000@st.com> <44C4A412.90200@st.com> In-Reply-To: <44C4A412.90200@st.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact binutils-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sourceware.org X-SW-Source: 2006-07/txt/msg00279.txt.bz2 Hi Andrew, >>> For reasons I do not understand I found that if I included the above >>> part of the patch to arch.exp I ended up with *more* SH linker >>> testsuite failures (for little endian multilibs) than without it. >> Hmmm, did you use both parts of the patch? Well I used my version of it, but essentially yes. > Well, I did the tests with and without my two-part patch. > > Without: > === ld Summary for sh-sim/-ml === > > # of expected passes 133 > # of unexpected failures 357 > # of expected failures 1 > # of unresolved testcases 14 > # of untested testcases 378 > With: > === ld Summary for sh-sim/-ml === > > # of expected passes 797 > # of unexpected failures 80 > # of expected failures 1 > # of untested testcases 21 > So that's quite considerably better. I have not looked at what causes > the 65 additional failures in little endian, This is where I thought that the second part of the patch was giving problems. Without it I get only 33 unexpected failures for the "-ml" multilib... I suspect however that this was connected to the fact that some architecture variants do not support little endian and the tests do not current make allowances for this. Anyway thanks very much for looking into this, and I am sure that at some date somebody will finish off the work and fix the remaining failures. Cheers Nick