From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 85441 invoked by alias); 4 Nov 2015 08:44:01 -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 85421 invoked by uid 89); 4 Nov 2015 08:44:00 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_HELO_PASS,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mail.anw.at Received: from ns1.anw.at (HELO mail.anw.at) (195.234.101.234) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Wed, 04 Nov 2015 08:43:58 +0000 Received: from [192.168.23.40] (anwhome.anw.at [195.234.103.23]) by mail.anw.at (8.14.4/8.14.4/Debian-4) with ESMTP id tA48htdd018982; Wed, 4 Nov 2015 09:43:55 +0100 Subject: Re: [PATCH] Add support for ARM embedded multilibs To: Ramana Radhakrishnan References: <56395178.8000006@anw.at> Cc: gcc-patches From: "Jasmin J." Message-ID: <5639C54F.5070700@anw.at> Date: Wed, 04 Nov 2015 08:44:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2015-11/txt/msg00298.txt.bz2 Hello Ramana! > Thank you for your patch - In this case before you make any more > changes to this patch - comparing your patch and Terry's patch here > https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00729.html shows no real > differences As I wrote in the patch, it is a port from the embedded-4_9-branch and it is exactly what Terry did. I didn't know this patch. I extracted it from the branch. All other stuff required to build a gcc for the ARM embedded CPUs are already on gcc trunk. This last piece is missing to get rid of the embedded-X_X-branch! I would love to see it in mainline GCC. > I would like to ask if you have a copyright assignment on file with the FSF I will do that. > How was your patch tested The patch is old an doesn't contain any real GCC change, but support for the library building process only. This was used since 15 months by many people and I used it to build an ARM embedded compiler based on gcc-5-branch. > see for example how I added t-aprofile to the backend and the kind of > testing it underwent I will look on that, if it is really required. >> * configure.ac (with_multilib_list): Export for being used in arm >> embedded multilib fragment. >> * Makefile.in (with_multilib_list): Import for being used in >> multilib fragment. > > This is already being used in config.gcc - why do you need this > additional hunk ? To be hones, I ported the patch and checked if it works. I will analyse it more detailed, if this is really required. > The t-rmprofile file will need updating to newer values for -mcpu and > march as well as comments up top to explain what multilibs are being > built . Where can I find them? BR Jasmin