Re: [PATCH] rs6000_adjust_cost old thinko

2015-11-30 Thread Eric Botcazou
> FYI, the function should test recog_memoized (dep_insn) also. I don't think that's needed as it doesn't call get_attr_type on dep_insn. -- Eric Botcazou

Re: [PATCH] rs6000_adjust_cost old thinko

2015-11-30 Thread Eric Botcazou
> Note this also is wrong on PA and one of the SPARC adjust_cost macros. Thanks for the heads up, fixed thusly, applied on the mainline PR target/28115 * config/sparc/sparc.c (supersparc_adjust_cost): Fix thinko. (sparc_adjust_cost): Add missing space. -- Eric

Re: [PATCH] rs6000_adjust_cost old thinko

2015-11-30 Thread David Edelsohn
On Mon, Nov 30, 2015 at 4:44 AM, Eric Botcazou wrote: >> Note this also is wrong on PA and one of the SPARC adjust_cost macros. > > Thanks for the heads up, fixed thusly, applied on the mainline > > > PR target/28115 > * config/sparc/sparc.c

[PATCH] rs6000_adjust_cost old thinko

2015-11-29 Thread David Edelsohn
When rs6000_adjust_cost originally was written, the return value from recog_memoized() was handled incorrectly. Note this also is wrong on PA and one of the SPARC adjust_cost macros. Bootstrapped on powrepc-ibm-aix7.1.0.0. PR target/28115 * config/rs6000/rs6000.c (rs6000_adjust_cost): Correct