On Tue, 2015-10-06 at 15:59 +0200, Bernd Schmidt wrote: > On 10/06/2015 03:37 PM, tbsaunde+...@tbsaunde.org wrote: > > This seems a bit cleaner, and should involve less allocation. > > I agree this is good. rtx_insn_list should die. > > > I tested there was no regressions for sh-sim with all languages accept > > ada,lto,fortran, ok? > > Could you please also build a few large source files and compare > before/after code generation to ensure there are no changes? I always > try to do this for patches like this one to avoid accidents, and this > change looks sufficiently non-mechanical to warrant it. Could also test > mips-elf.
I've ran a CSiBE comparison for "-O2 -m4-single -ml" on sh-elf. There are no code changes with Trev's patch. Cheers, Oleg