From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27995 invoked by alias); 31 May 2012 22:48:03 -0000 Received: (qmail 27827 invoked by uid 22791); 31 May 2012 22:48:01 -0000 X-SWARE-Spam-Status: No, hits=-4.6 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,KHOP_RCVD_TRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE X-Spam-Check-By: sourceware.org Received: from mail-pz0-f47.google.com (HELO mail-pz0-f47.google.com) (209.85.210.47) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 31 May 2012 22:47:48 +0000 Received: by dalh21 with SMTP id h21so1881892dal.20 for ; Thu, 31 May 2012 15:47:47 -0700 (PDT) Received: by 10.68.217.138 with SMTP id oy10mr4179124pbc.30.1338504467205; Thu, 31 May 2012 15:47:47 -0700 (PDT) MIME-Version: 1.0 Received: by 10.68.43.102 with HTTP; Thu, 31 May 2012 15:47:27 -0700 (PDT) In-Reply-To: <87ipfcf2yy.fsf@talisman.home> References: <1330141994-26174-1-git-send-email-mattst88@gmail.com> <8762ev1ge4.fsf@talisman.home> <87ipfcf2yy.fsf@talisman.home> From: Matt Turner Date: Thu, 31 May 2012 22:48:00 -0000 Message-ID: Subject: Re: [PATCH 1/2] mips: Add R4600 scheduling support for imul and idiv To: gcc-patches@gcc.gnu.org, Eric Christopher , rdsandiford@googlemail.com Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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: 2012-05/txt/msg02130.txt.bz2 On Thu, May 31, 2012 at 5:35 PM, Richard Sandiford wrote: > Matt Turner writes: >> On Sat, Feb 25, 2012 at 3:11 AM, Richard Sandiford >> wrote: >>> Matt Turner writes: >>>> The r4600_imul and r4600_idiv reservations were correct for si, but >>>> there were no *_di reservations. >>>> >>>> See page 4 of >>>> http://www.sgistuff.net/hardware/other/documents/R4600_Prod_OV.pdf >>>> >>>> 2012-02-24 =A0Matt Turner =A0 >>>> >>>> =A0 =A0 =A0 * config/mips/4600.md (r4600_imul_si): Rename from r4600_i= mul. >>>> =A0 =A0 =A0 (r4600_imul_di): New. >>>> =A0 =A0 =A0 (r4600_idiv_si): Rename from r4600_idiv. >>>> =A0 =A0 =A0 (r4600_idiv_di): New. >>> >>> Both patches look good, thanks. =A0Will commit once 4.8 is open and the >>> copyright assignment is sorted. >>> >>> Richard >> >> Copyright assignment is sorted. Please commit. :) > > Applied this one. =A0Part 2 seems to be based on a different version > of driver-native.c though. > > Thanks for perservering. :-) > > Richard Thanks a lot! Ah, right, 2/2 was written before IRIX support was removed and changed driver-native.c significantly. Updated patch in your inbox shortly. Thanks! Matt