Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-09 Thread Jeff Law
On 10/08/13 21:39, Zhenqiang Chen wrote: I'll go ahead and pull the common bits into a single function and commit on Zhenqiang's behalf. Thank you! No problem. If you think you'll be contributing regularly, you might want to go ahead and ask for write-after-approval privileges. jeff

Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-09 Thread Jeff Law
On 09/30/13 03:29, Zhenqiang Chen wrote: Hi, The patch enhances phiopt to handle cases like: if (a == 0 && (...)) return 0; return a; Boot strap and no make check regression on X86-64 and ARM. Is it OK for trunk? Thanks! -Zhenqiang ChangeLog: 2013-09-30 Zhenqiang Chen

RE: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-08 Thread Zhenqiang Chen
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: Wednesday, October 09, 2013 5:00 AM > To: Andrew Pinski; Zhenqiang Chen > Cc: GCC Patches > Subject: Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR > > On 09/30/13 09:57, Andrew Pinski w

Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-10-08 Thread Jeff Law
On 09/30/13 09:57, Andrew Pinski wrote: On Mon, Sep 30, 2013 at 2:29 AM, Zhenqiang Chen wrote: Hi, The patch enhances phiopt to handle cases like: if (a == 0 && (...)) return 0; return a; Boot strap and no make check regression on X86-64 and ARM. Is it OK for trunk? From someo

Re: [PATCH] Enhance phiopt to handle BIT_AND_EXPR

2013-09-30 Thread Andrew Pinski
On Mon, Sep 30, 2013 at 2:29 AM, Zhenqiang Chen wrote: > Hi, > > The patch enhances phiopt to handle cases like: > > if (a == 0 && (...)) > return 0; > return a; > > Boot strap and no make check regression on X86-64 and ARM. > > Is it OK for trunk? >From someone who wrote lot of this code