This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Remove dead labels to increase superblock scope
- From: Eric Botcazou <ebotcazou at adacore dot com>
- To: Richard Sandiford <rdsandiford at googlemail dot com>
- Cc: Tom de Vries <Tom_deVries at mentor dot com>, Steven Bosscher <stevenb dot gcc at gmail dot com>, Michael Matz <matz at suse dot de>, gcc-patches at gcc dot gnu dot org
- Date: Sun, 4 Dec 2011 16:21:23 +0100
- Subject: Re: [PATCH] Remove dead labels to increase superblock scope
- References: <4EC65977.4020501@mentor.com> <4EDAB22E.2080204@mentor.com> <87fwh0ecfe.fsf@firetop.home>
> Looks good codewise.
Seconded, modulo the file: the function should be in cfgrtl.c instead.
> I'm just a bit worried about the name "delete_label".
> "delete_insn (label)" should always do the right thing for a pure deletion;
> the point of the new routine is that it also moves instructions.
It only fixes things up though, so that the RTL stream is valid again. Hence
the question: why not retrofit it into delete_insn directly?
--
Eric Botcazou