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 :).