On Mon, 2015-01-05 at 12:48 -0700, Jeff Law wrote: > On 01/03/15 05:18, Oleg Endo wrote: > > https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01730.html > > > > On Sun, 2014-12-21 at 23:02 +0100, Oleg Endo wrote: > >> Hi, > >> > >> It seems that quite some callers of refers_to_regno_p are interested in > >> testing a single register number only. The caller code can be > >> simplified by adding an overload for refers_to_regno_p. This is what > >> the patch does. No functional changes. Tested with 'make all-gcc' on > >> sh-elf cross config and bootstrapped on i686-pc-linux-gnu. > >> OK for trunk? > >> > >> Cheers, > >> Oleg > >> > >> gcc/ChangeLog: > >> * rtlanal.c (refers_to_regno_p): Change return value from int to > >> bool. > >> * rtl.h (refers_to_regno_p): Add overload. > >> * cse.c: Use it. > >> * bt-load.c: Likewise. > >> * combine.c: Likewise. > >> * df-scan.c: Likewise. > >> * sched-deps.c: Likewise. > >> * config/s390/s390.c: Likewise. > >> * config/m32r/m32r.c: Likewise. > >> * config/rs6000/spe.md: Likewise. > >> * config/rs6000/rs6000.c: Likewise. > >> * config/pa/pa.c: Likewise. > >> * config/stormy16/stormy16.c: Likewise. > >> * config/cris/cris.c: Likewise. > >> * config/arc/arc.md: Likewise. > >> * config/arc/arc.c: Likewise. > >> * config/sh/sh.md: Likewise. > >> * config/sh/sh.c: Likewise. > >> * config/frv/frv.c: Likewise. > OK for the trunk.
Thanks, committed as r219203. > Though I think further patches of this nature should > probably wait until stage1 reopens. Also for non-functional stuff like changing more return values / args from int to bool where it's obvious? Cheers, Oleg
