Thanks for the comments. Patch is updated.

-Zhenqiang

> -----Original Message-----
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Richard Henderson
> Sent: Saturday, October 11, 2014 11:03 PM
> To: Zhenqiang Chen; gcc-patches@gcc.gnu.org
> Subject: Re: [Ping] [PATCH, 2/10] prepare ccmp
> 
> On 09/22/2014 11:43 PM, Zhenqiang Chen wrote:
> > +     /* If jumps are cheap and the target does not support conditional
> > +        compare, turn some more codes into jumpy sequences.  */
> > +     else if (BRANCH_COST (optimize_insn_for_speed_p (), false) < 4
> > +              && (targetm.gen_ccmp_first == NULL))
> 
> Don't add unnecessary parenthesis around the == expression.
> 
> Otherwise ok.
> 
> 
> r~

Attachment: 2-prepare.patch
Description: Binary data

Reply via email to