This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

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

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Oleg Endo <oleg dot endo at t-online dot de>
Cc: tbsaunde+gcc at tbsaunde dot org, gcc-patches at gcc dot gnu dot org
Date: Tue, 6 Oct 2015 16:57:27 +0200
Subject: Re: [PATCH] reorg.c: use vec<rtx_insn *> instead of rtx_insn_list for the delay insn list
Authentication-results: sourceware.org; auth=none
References: <1444138640-4331-1-git-send-email-tbsaunde+gcc at tbsaunde dot org> <5613D3D0 dot 7000309 at redhat dot com> <1444143044 dot 2031 dot 668 dot camel at t-online dot de>

On 10/06/2015 04:50 PM, Oleg Endo wrote:

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.


Thanks! Good enough.


Bernd

References:
- [PATCH] reorg.c: use vec<rtx_insn *> instead of rtx_insn_list for the delay insn list
  - From: tbsaunde+gcc
- Re: [PATCH] reorg.c: use vec<rtx_insn *> instead of rtx_insn_list for the delay insn list
  - From: Bernd Schmidt
- Re: [PATCH] reorg.c: use vec<rtx_insn *> instead of rtx_insn_list for the delay insn list
  - From: Oleg Endo

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]