From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 29156 invoked by alias); 10 Aug 2007 15:41:40 -0000 Received: (qmail 29097 invoked by uid 22791); 10 Aug 2007 15:41:40 -0000 X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 10 Aug 2007 15:41:29 +0000 Received: (qmail 31883 invoked from network); 10 Aug 2007 15:41:27 -0000 Received: from unknown (HELO gateway) (10.0.0.100) by mail.codesourcery.com with SMTP; 10 Aug 2007 15:41:27 -0000 Received: by gateway (Postfix, from userid 1010) id 8F3BE6C0CF; Fri, 10 Aug 2007 08:41:27 -0700 (PDT) From: Richard Sandiford To: Nick Clifton Mail-Followup-To: Nick Clifton ,echristo@apple.com, gcc-patches@gcc.gnu.org, richard@codesourcery.com Cc: echristo@apple.com, gcc-patches@gcc.gnu.org Subject: Re: RFA/RFC: GCC has an incorrect length for the ldc1 instruction References: <87hcnb8ybn.fsf@firetop.home> <46BC103B.4090007@redhat.com> <87vebnddn2.fsf@firetop.home> Date: Fri, 10 Aug 2007 15:41:00 -0000 In-Reply-To: <87vebnddn2.fsf@firetop.home> (Richard Sandiford's message of "Fri\, 10 Aug 2007 08\:58\:41 +0100") Message-ID: <87zm0zbdnd.fsf@firetop.home> User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii 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: 2007-08/txt/msg00673.txt.bz2 Richard Sandiford writes: > Nick Clifton writes: >> Hi Richard, >>> gcc/ >>> * config/mips/mips-protos.h (mips_address_insns): Add a boolean >>> argument. >>> (mips_fetch_insns): Delete in favor of... >>> (mips_load_store_insns): ...this new function. >>> * config/mips/mips.c (mips_address_insns): Add a boolean argument >>> to say whether multiword moves _might_ be split. >>> (mips_fetch_insns): Delete in favor of... >>> (mips_load_store_insns): ...this new function. >>> (mips_rtx_costs): Update the call to mips_address_insns. >>> (mips_address_cost): Likewise. >>> * config/mips/mips.md (length): Use mips_load_store_insns instead >>> of mips_fetch_insns. >>> * config/mips/constraints.md (R): Use mips_address_insns rather >>> than mips_fetch_insns. Assume that the move never needs to be split. >> >> Are you going to apply this version of the patch, or may I do it on >> your behalf ? > > Sorry for the delay. I've been really busy recently, and only started > running tests for the patch yesterday. I'll commit it today or tomorrow > if the results are OK. Now committed after testing on mipsisa32r2-elfoabi. Richard