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.