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."

Reply via email to