Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-03-04 Thread H.J. Lu
On Sat, Mar 1, 2014 at 2:19 AM, Richard Biener wrote: > On Fri, Feb 28, 2014 at 7:23 PM, H.J. Lu wrote: >> On Fri, Feb 28, 2014 at 9:42 AM, H.J. Lu wrote: >>> On Fri, Feb 28, 2014 at 9:25 AM, H.J. Lu wrote: On Fri, Feb 28, 2014 at 8:11 AM, H.J. Lu wrote: > On Fri, Feb 28, 2014 at 2:09

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-03-01 Thread Bin.Cheng
Hi H.J, Sorry that I will be out of office next week, and don't have chance to reproduce it until back. BTW, does x32 refer to x86 32 bit? Thanks, bin On Sat, Mar 1, 2014 at 2:23 AM, H.J. Lu wrote: > On Fri, Feb 28, 2014 at 9:42 AM, H.J. Lu wrote: >> On Fri, Feb 28, 2014 at 9:25 AM, H.J. Lu w

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-03-01 Thread Richard Biener
On Fri, Feb 28, 2014 at 7:23 PM, H.J. Lu wrote: > On Fri, Feb 28, 2014 at 9:42 AM, H.J. Lu wrote: >> On Fri, Feb 28, 2014 at 9:25 AM, H.J. Lu wrote: >>> On Fri, Feb 28, 2014 at 8:11 AM, H.J. Lu wrote: On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener wrote: > On Fri, Feb 28, 2014 a

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-28 Thread H.J. Lu
On Fri, Feb 28, 2014 at 9:42 AM, H.J. Lu wrote: > On Fri, Feb 28, 2014 at 9:25 AM, H.J. Lu wrote: >> On Fri, Feb 28, 2014 at 8:11 AM, H.J. Lu wrote: >>> On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener >>> wrote: On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener wrote: > On Fri, Fe

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-28 Thread H.J. Lu
On Fri, Feb 28, 2014 at 9:25 AM, H.J. Lu wrote: > On Fri, Feb 28, 2014 at 8:11 AM, H.J. Lu wrote: >> On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener >> wrote: >>> On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener >>> wrote: On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu wrote: > On Mon, Feb

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-28 Thread H.J. Lu
On Fri, Feb 28, 2014 at 8:11 AM, H.J. Lu wrote: > On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener > wrote: >> On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener >> wrote: >>> On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu wrote: On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng wrote: > Hi, > Th

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-28 Thread H.J. Lu
On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener wrote: > On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener > wrote: >> On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu wrote: >>> On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng wrote: Hi, This patch is to fix regression reported in PR60280 by removi

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-28 Thread Richard Biener
On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener wrote: > On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu wrote: >> On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng wrote: >>> Hi, >>> This patch is to fix regression reported in PR60280 by removing forward loop >>> headers/latches in cfg cleanup if possible.

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-28 Thread Richard Biener
On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu wrote: > On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng wrote: >> Hi, >> This patch is to fix regression reported in PR60280 by removing forward loop >> headers/latches in cfg cleanup if possible. Several tests are broken by >> this change since cfg cleanup i

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread Bin.Cheng
Sorry, I didn't test it against logical_op_short_circuit target. I will look into this PR. Thanks, bin On Fri, Feb 28, 2014 at 9:34 AM, Hans-Peter Nilsson wrote: > On Tue, 25 Feb 2014, bin.cheng wrote: > >> Hi, >> This patch is to fix regression reported in PR60280 by removing forward loop >> h

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread Bin.Cheng
Thanks for reporting this, I will look into it. Thanks, bin On Fri, Feb 28, 2014 at 8:52 AM, H.J. Lu wrote: > On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng wrote: >> Hi, >> This patch is to fix regression reported in PR60280 by removing forward loop >> headers/latches in cfg cleanup if possible.

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread Hans-Peter Nilsson
On Tue, 25 Feb 2014, bin.cheng wrote: > Hi, > This patch is to fix regression reported in PR60280 by removing forward loop > headers/latches in cfg cleanup if possible. Several tests are broken by > this change since cfg cleanup is shared by all optimizers. Some tests has > already been fixed by

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread H.J. Lu
On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng wrote: > Hi, > This patch is to fix regression reported in PR60280 by removing forward loop > headers/latches in cfg cleanup if possible. Several tests are broken by > this change since cfg cleanup is shared by all optimizers. Some tests has > already b

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-25 Thread Richard Biener
Cheng >> Cc: GCC Patches >> Subject: Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop >> preheaders and latches >> >> On Tue, Feb 25, 2014 at 6:12 AM, bin.cheng wrote: >> > Hi, >> > This patch is to fix regression reported in PR60280 by removi

RE: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-25 Thread bin.cheng
Updated as comments. Thanks, bin > -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: Tuesday, February 25, 2014 6:38 PM > To: Bin Cheng > Cc: GCC Patches > Subject: Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop > p

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-25 Thread Richard Biener
On Tue, Feb 25, 2014 at 6:12 AM, bin.cheng wrote: > Hi, > This patch is to fix regression reported in PR60280 by removing forward loop > headers/latches in cfg cleanup if possible. Several tests are broken by > this change since cfg cleanup is shared by all optimizers. Some tests has > already b

[PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-24 Thread bin.cheng
Hi, This patch is to fix regression reported in PR60280 by removing forward loop headers/latches in cfg cleanup if possible. Several tests are broken by this change since cfg cleanup is shared by all optimizers. Some tests has already been fixed by recent patches, I went through and fixed the oth