Georg-Johann Lay <a...@gjlay.de> writes: > @@ -1257,17 +1129,19 @@ print_operand_address (FILE *file, rtx a > rtx x = addr; > if (GET_CODE (x) == CONST) > x = XEXP (x, 0); > - if (GET_CODE (x) == PLUS && GET_CODE (XEXP (x,1)) == CONST_INT) > + if (GET_CODE (x) == PLUS > + && CONST_INT_P (XEXP (x,1)))
Please also add a space after the comma. > @@ -5034,9 +4933,11 @@ avr_insert_attributes (tree node, tree * > { > static const char dsec[] = ".progmem.data"; > > - *attributes = tree_cons (get_identifier ("section"), > - build_tree_list (NULL, build_string > (strlen (dsec), dsec)), > - *attributes); > + *attributes = > + tree_cons (get_identifier ("section"), > + build_tree_list (NULL, > + build_string (strlen (dsec), dsec)), > + *attributes); GNU style is to break before the operator. Andreas. -- Andreas Schwab, sch...@redhat.com GPG Key fingerprint = D4E8 DBE3 3813 BB5D FA84 5EC7 45C6 250E 6F00 984E "And now for something completely different."