>   This patch remove some unused macros from sparc.h. The
> RTX_OK_FOR_OFFSET_P and RTX_OK_FOR_OLO10_P macros is used only in
> sparc_legitimate_address_p function and moved to sparc.c.

Thanks for spotting this.

>   OK to install?
>
>
>         * config/sparc/sparc.h (REG_OK_FOR_INDEX_P, REG_OK_FOR_BASE_P,
>         SYMBOLIC_CONST, RTX_OK_FOR_BASE_P, RTX_OK_FOR_INDEX_P): Remove.
>         (RTX_OK_FOR_OFFSET_P, RTX_OK_FOR_OLO10_P): Move to...
>         * config/sparc/sparc.c (RTX_OK_FOR_OFFSET_P,
>         RTX_OK_FOR_OLO10_P): ...here.
>         (sparc_mode_dependent_address_p): Use symbolic_operand instead of
>         SYMBOLIC_CONST.

OK if you move...

> On SPARC, the actual legitimate addresses must be REG+REG or
> REG+SMALLINT -   ordinarily.  This changes a bit when generating PIC.

...this couple of sentences to the head comment of sparc_legitimate_address_p.

-- 
Eric Botcazou

Reply via email to