On 06/01/12 04:57, Richard Henderson wrote:
> Ping.  Reposting the patch as it required updates to HEAD.
> 
> 
> r~
> 
> 

This is OK apart from:

> +       gcc_assert(!reload_completed);

Missing space.


>  (define_special_predicate "add_operator"
>    (match_code "plus"))
> +
> +(define_predicate "mem_noofs_operand"
> +  (and (match_code "mem")
> +       (match_code "reg" "0")))


Hmm, I'm a bit worried about things like the frame pointer reg matching
this and then everything going screwy when elimination takes place.
Almost certainly LEGITIMIZE_RELOAD_ADDRESS is going to be confused here
as it doesn't have enough information to know what's legitimate in this
particular case.

R.

Reply via email to