This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: more new code for reorg.c (second try)
- To: Herman ten Brugge <Haj dot Ten dot Brugge at net dot HCC dot nl>
- Subject: Re: more new code for reorg.c (second try)
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Mon, 29 Nov 1999 01:17:25 -0700
- cc: egcs-patches at egcs dot cygnus dot com
- Reply-To: law at cygnus dot com
In message <199910310857.IAA03682@net.HCC.nl>you write:
> Hello,
>
> I generated some new code for reorg.c to handle targets with more than
> 1 delay slot. Currently only the c4x (3 delay slots) has these. I know
> some one who is working on the sharc port (2 delay slots) for gcc that
> also uses these patches.
> I optimized the routine fill_eager_delay_slots to try to extend an
> existing delay list.
> I had to modify emit_delay_sequence to handle a SEQUENCE. This was
> the best way to extend an existing delay list.
> I had to modify fix_reg_dead_note to handle sequences correctly so
> I added a new helper routine called next_seq_nonnote_insn.
You still have not explained *why* you need to handle a SEQUENCE in
emit_delay_sequence. And as I've explained in the past, I think that is
wrong and is just papering over problems elsewhere. Until you provide
details about why this needed this patch will not be installed.
jeff