From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30218 invoked by alias); 14 Oct 2003 08:07:51 -0000 Mailing-List: contact gcc-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-owner@gcc.gnu.org Received: (qmail 30209 invoked from network); 14 Oct 2003 08:07:48 -0000 Received: from unknown (HELO develer.com) (151.38.19.110) by sources.redhat.com with SMTP; 14 Oct 2003 08:07:48 -0000 Received: (qmail 14873 invoked from network); 14 Oct 2003 08:07:43 -0000 Received: from beetle.trilan (HELO develer.com) (?jzWjV3eYMLdGPQG6MlYUt7JND7gRinTd?@10.3.3.220) by ns.trilan with SMTP; 14 Oct 2003 08:07:43 -0000 Message-ID: <3F8BAED0.70004@develer.com> Date: Tue, 14 Oct 2003 12:40:00 -0000 From: Bernardo Innocenti Organization: Develer S.r.l. User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20030927 X-Accept-Language: en, en-us MIME-Version: 1.0 To: Gunther Nikl CC: GCC Mailing List , Richard Henderson Subject: Re: Testing m68k changes on AmigaOS and Linux/m68k References: <3F88BBCF.7090106@develer.com> <20031013152223.GA45308@lorien.int.gecko.de> In-Reply-To: <20031013152223.GA45308@lorien.int.gecko.de> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-10/txt/msg00492.txt.bz2 Gunther Nikl wrote: >>Since I can only test on ColdFire targets, would you mind doing a >>bootstrap on Linux/m68k and/or AmigaOS? > > I maintain my patches on a cross-setup by building a cross-compiler. > That cross-compiler is then used to build a "native" compiler for my > target. I don't boostrap on the target because thats much to slow for > my taste ;-( I guess building on my old Amiga 4000 with its 25MHz 68040 would take a couple of days :-) > Building the cross-compiler revealed two errors in m68k.c. The patch is > attached. Using the cross-compiler to built the native compiler revealed > two bugs (?) in gcc/Makefile[.in]. You're probably one of the few people who exercise canadian cross builds :-) > I changed the Makefile and was able > to build the native compiler but I don't know whether my changes to the > Makefile are correct. The cross-compiler to AmigaOS seems to work > correctly (modulo the open issues regarding small-data). I goint to test > the coss-built native compiler tonight. I see you're using the MIT syntax on the Amiga. Some guy told me the GeekGadgets port of GCC used it. Have you ever tried defining MOTOROLA? If you're luckly, it should work out of the box. I'm asking because I still have a (not so hidden) agenda for obsoleting the MIT syntax some day... Your patch looks fine to me, but it doesn't fall under the "obvious patch" definition (it changes code). Richard, is it OK to apply? > --cut-- > 2003-10-13 Gunther Nikl > > * config/m68k/m68k.c (m68k_output_function_prologue): Fix usage of > current_frame at two places (one type and one missing) > > --- m68k.c.orig Mon Oct 13 11:35:20 2003 > +++ m68k.c Mon Oct 13 14:48:23 2003 > @@ -623,7 +623,7 @@ > #ifdef MOTOROLA > asm_fprintf (stream, "\tfmovm %I0x%x,-(%Rsp)\n", current_frame.fpu_mask); > #else > - asm_fprintf (stream, "\tfmovem %I0x%x,%Rsp@-\n", current_frmae.fpu_mask); > + asm_fprintf (stream, "\tfmovem %I0x%x,%Rsp@-\n", current_frame.fpu_mask); > #endif > if (dwarf2out_do_frame ()) > { > @@ -934,7 +934,7 @@ > #else > asm_fprintf (stream, "\tmoveml %s@(-%wd),%I0x%x\n", > reg_names[FRAME_POINTER_REGNUM], > - offset + fsize, > + current_frame.offset + fsize, > current_frame.reg_mask); > #endif > } > --cut-- -- // Bernardo Innocenti - Develer S.r.l., R&D dept. \X/ http://www.develer.com/ Please don't send Word attachments - http://www.gnu.org/philosophy/no-word-attachments.html