[PATCH] reorg.c: use vec<rtx_insn *> instead of rtx_insn_list for the delay insn list

Oleg Endo oleg.endo@t-online.de
Tue Oct 6 14:51:00 GMT 2015


On Tue, 2015-10-06 at 15:59 +0200, Bernd Schmidt wrote:
> On 10/06/2015 03:37 PM, tbsaunde+gcc@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




More information about the Gcc-patches mailing list