From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31544 invoked by alias); 28 Nov 2009 00:49:51 -0000 Received: (qmail 31536 invoked by uid 22791); 28 Nov 2009 00:49:51 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL,BAYES_00,SARE_MSGID_LONG40,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mail-ew0-f227.google.com (HELO mail-ew0-f227.google.com) (209.85.219.227) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sat, 28 Nov 2009 00:49:47 +0000 Received: by ewy27 with SMTP id 27so2489700ewy.16 for ; Fri, 27 Nov 2009 16:49:45 -0800 (PST) MIME-Version: 1.0 Received: by 10.216.87.11 with SMTP id x11mr573038wee.16.1259369383046; Fri, 27 Nov 2009 16:49:43 -0800 (PST) In-Reply-To: <4AFC5322.1010308@t-online.de> References: <4AD9CEF2.50908@t-online.de> <200910221504.37462.ebotcazou@adacore.com> <4AFC5322.1010308@t-online.de> Date: Sat, 28 Nov 2009 01:39:00 -0000 Message-ID: <6dc9ffc80911271649j76d5a78x15c6cb11683907f7@mail.gmail.com> Subject: Re: regrename speedup From: "H.J. Lu" To: Bernd Schmidt Cc: Eric Botcazou , gcc-patches@gcc.gnu.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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: 2009-11/txt/msg01572.txt.bz2 On Thu, Nov 12, 2009 at 10:25 AM, Bernd Schmidt wr= ote: > Eric Botcazou wrote: >>> =A0 =A0 =A0PR rtl/38582 >> >> PR rtl-opt (or PR rtl-optimization). >> >>> =A0 =A0 =A0* regrename.c (struct du_head): New structure; some elements= moved >>> =A0 =A0 =A0from... >>> =A0 =A0 =A0(struct du_chain): ... this one. >>> =A0 =A0 =A0(open_chains, closed_chains): Now of type struct du_head *. >>> =A0 =A0 =A0(do_replace): Accept du_head argument, not du_chain. =A0All = callers >>> =A0 =A0 =A0changed. =A0Modified code to match new data structures. >>> =A0 =A0 =A0(build_def_use): Return a list of du_head structures. =A0Mod= ified code >>> =A0 =A0 =A0to match new data structures. >>> =A0 =A0 =A0(dump_def_use_chain): Accept du_head argument, not du_chain.= =A0All >>> =A0 =A0 =A0callers changed. =A0Modified code to match new data structur= es. >>> =A0 =A0 =A0(merge_overlapping_regs): Accept du_head argument, not du_ch= ain. =A0All >>> =A0 =A0 =A0callers changed. =A0Modified code to match new data structur= es. >>> =A0 =A0 =A0(scan_rtx_reg): Change type of this_regno and this_nregs to = unsigned. >>> =A0 =A0 =A0Allocate a du_head structure as well as a du_chain when crea= ting a >>> =A0 =A0 =A0new chain. =A0Modified other code to match new data structur= es. >> >> OK with the trailing spaces removed in > [...] > > Thanks, committed now (vacation got in the way). =A0I'll post a followup > patch as well which takes care of the remaining performance problem. > > This caused: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D42202 --=20 H.J.