From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17749 invoked by alias); 28 Jan 2013 21:44:31 -0000 Received: (qmail 17740 invoked by uid 22791); 28 Jan 2013 21:44:30 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,KHOP_SPAMHAUS_DROP,KHOP_THREADED,RCVD_IN_DNSWL_NONE,RCVD_IN_HOSTKARMA_NO,RCVD_IN_HOSTKARMA_YE X-Spam-Check-By: sourceware.org Received: from caibbdcaaaaf.dreamhost.com (HELO homiemail-a93.g.dreamhost.com) (208.113.200.5) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 28 Jan 2013 21:43:56 +0000 Received: from homiemail-a93.g.dreamhost.com (localhost [127.0.0.1]) by homiemail-a93.g.dreamhost.com (Postfix) with ESMTP id E55428405C; Mon, 28 Jan 2013 13:43:55 -0800 (PST) Received: from AMDP6 (81-234-195-35-no56.tbcn.telia.com [81.234.195.35]) (Authenticated sender: rdos@rdos.net) by homiemail-a93.g.dreamhost.com (Postfix) with ESMTPA id 3069184059; Mon, 28 Jan 2013 13:43:53 -0800 (PST) Message-ID: From: "Leif Ekblad" To: "Uros Bizjak" Cc: "Richard Biener" , , "H.J. Lu" , "Jakub Jelinek" References: <4A4D1EB90016420C88D9C8DF7B633E73@AMDP6> <15368AFB015C4FB9B1FB9B080B43DBE8@AMDP6> <329FEE5DE1D3490D85226F0FF93154F2@AMDP6> <123C318BEB9645F6A2E41B86E3648CED@AMDP6> Subject: Re: [PATCH] Adding target rdos to GCC Date: Mon, 28 Jan 2013 21:44:00 -0000 MIME-Version: 1.0 Content-Type: text/plain; format=flowed; charset="iso-8859-1"; reply-type=original Content-Transfer-Encoding: 7bit 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: 2013-01/txt/msg01353.txt.bz2 That looks good. Thanks, Uros. Leif ----- Original Message ----- From: "Uros Bizjak" To: "Leif Ekblad" Cc: "Richard Biener" ; ; "H.J. Lu" ; "Jakub Jelinek" Sent: Monday, January 28, 2013 9:45 PM Subject: Re: [PATCH] Adding target rdos to GCC > On Mon, Jan 28, 2013 at 9:14 PM, Leif Ekblad wrote: > >>>> That is intentional. The gthr-rdos.h file is part of libgcc. My >>>> intention >>>> was to first patch gcc, then update the patches for newlib, and finally >>>> libgcc. The gthr-rdos.h file would reference include-files part of >>>> newlib, >>>> so this is kind of circular. I also cannot define the thread model for >>>> RDOS >>>> unless I define this file. >>>> >>>> I see a couple of possible solutions: >>>> 1. Keep as is. You cannot build libgcc at the current stage anyway, and >>>> the >>>> bootstrap must be built without threading >>>> 2. Add an empty gthr-rdos.h file until libgcc is done >>>> 3. Remove the threading-model for now, and add it with libgcc instead. >>> >>> >>> I propose option 3. >>> >>> Is it enough to remove gthr.m4 change from the patch in this case? >> >> Yes, for all practical purposes. There is a reference to thread-file in >> config.gcc, when threading is enabled, which doesn't work for >> bootstrapping >> the compiler anyway. > > Thanks for pointing it, I have also removed this reference. > > Attached is the patch that has been committed to SVN. I have added > missing licence headers to new files and clean whitespace a bit. > > 2013-01-28 Leif Ekblad > > * config.gcc (i[34567]86-*-rdos*, x86_64-*-rdos*): New targets. > * config/i386/i386.h (TARGET_RDOS): New macro. > (DEFAULT_LARGE_SECTION_THRESHOLD): New macro. > * config/i386/i386.c (ix86_option_override_internal): For 64bit > TARGET_RDOS, set ix86_cmodel to CM_MEDIUM_PIC and flag_pic to 1. > * config/i386/i386.opt (mlarge-data-threshold): Initialize to > DEFAULT_LARGE_SECTION_THRESHOLD. > * config/i386/i386.md (R14_REG, R15_REG): New constants. > * config/i386/rdos.h: New file. > * config/i386/rdos64.h: New file. > > Thanks, > Uros. >