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 hjl.to...@gmail.com wrote: On Fri, Feb 28, 2014 at 9:42 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri,

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 hjl.to...@gmail.com wrote: On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng bin.ch...@arm.com 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

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 richard.guent...@gmail.com wrote: On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng bin.ch...@arm.com wrote: Hi, This patch is to fix regression reported in PR60280 by removing forward

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 richard.guent...@gmail.com wrote: On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng bin.ch...@arm.com

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 hjl.to...@gmail.com wrote: On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Feb 28, 2014 at 1:52 AM, H.J. Lu hjl.to...@gmail.com

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 hjl.to...@gmail.com wrote: On Fri, Feb 28, 2014 at 8:11 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Feb 28, 2014 at 10:09 AM, Richard Biener richard.guent...@gmail.com

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 hjl.to...@gmail.com wrote: On Fri, Feb 28, 2014 at 9:25 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Feb 28, 2014 at 8:11 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, Feb 28, 2014 at 2:09 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri,

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 bin.ch...@arm.com 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

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 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 hjl.to...@gmail.com wrote: On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng bin.ch...@arm.com wrote: Hi, This patch is to fix regression reported in PR60280 by removing forward loop headers/latches

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 h...@bitrange.com wrote: On Tue, 25 Feb 2014, bin.cheng wrote: Hi, This patch is to fix regression reported in PR60280 by removing

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 bin.ch...@arm.com 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

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 preheaders and latches On Tue

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

2014-02-25 Thread Richard Biener
: Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches On Tue, Feb 25, 2014 at 6:12 AM, bin.cheng bin.ch...@arm.com wrote: Hi, This patch is to fix regression reported in PR60280 by removing forward loop headers/latches in cfg cleanup if possible. Several tests