From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3968 invoked by alias); 6 Jan 2007 13:45:55 -0000 Received: (qmail 3959 invoked by uid 22791); 6 Jan 2007 13:45:55 -0000 X-Spam-Check-By: sourceware.org Received: from nf-out-0910.google.com (HELO nf-out-0910.google.com) (64.233.182.189) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sat, 06 Jan 2007 13:45:51 +0000 Received: by nf-out-0910.google.com with SMTP id a25so451711nfc for ; Sat, 06 Jan 2007 05:45:48 -0800 (PST) Received: by 10.82.111.8 with SMTP id j8mr2348879buc.1168091148499; Sat, 06 Jan 2007 05:45:48 -0800 (PST) Received: by 10.82.150.13 with HTTP; Sat, 6 Jan 2007 05:45:48 -0800 (PST) Message-ID: <84fc9c000701060545x1952b2a6x4d4dad8ebf8e7313@mail.gmail.com> Date: Sat, 06 Jan 2007 13:45:00 -0000 From: "Richard Guenther" To: "Steven Bosscher" Subject: Re: [PATCH] Clean up ifcvt.c a bit Cc: gcc-patches@gcc.gnu.org, "Ian Lance Taylor" In-Reply-To: <200701061425.05608.steven.bosscher@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <200701061425.05608.steven.bosscher@gmail.com> 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: 2007-01/txt/msg00461.txt.bz2 On 1/6/07, Steven Bosscher wrote: > Hello, > > This is the first of (probably) a few patches to clean up ifcvt.c to > make it ready for working in cfglayout mode. > > The first part of this patch just factors nearly identical code into > a new function. > The second part stops us from copying global_live_at_end. We don't > have to do this, because merge_blocks already takes care of it. > > Bootstrapped (c,c++,objc,fortran) + tested on x86-64-unknown-linux-gnu. > OK for trunk? This is ok. Thanks, Richard. > Gr. > Steven > > * ifcvt.c (cond_move_convert_if_block): New function, code > factored out from... > (cond_move_process_if_block): ...here. Call the new function > on the THEN and ELSE blocks. > (merge_if_block): Do not copy global_live_at_end, merge_blocks > already takes care of this. >