https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93131

--- Comment #9 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #8)
> 
> Formatting, s/(/ (/g
> 
> >    }
> >    (if (wi::eq(mask & cst1,mask & cst2))
> 
> Ditto, also add space before ,
> 
> >     (cmp (bit_and @0 {wide_int_to_tree (type1, mask); } ) { wide_int_to_tree
> 
> And here add space before wide_int_to_tree.
> 
> > (cst1|cst2, mask); } )
> 
> And these arguments need to be (type1, cst1 | cst2), right?
> The line is too long, so maybe put the second { wide_int... on another line
> indented below (bit_and ?
> 
> >     { constant_boolean_node (cmp == EQ_EXPR, type); }))))

I wrote that directly in the bug report :).  Yes I Know the formating is wrong.
 and looks like I messed up the arguments to the second wide_int_to_tree too
:).

Reply via email to