This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR 50826] [DF] no debug temps for artificial uses
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Alexandre Oliva <aoliva at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 9 Apr 2012 12:57:38 +0200
- Subject: Re: [PR 50826] [DF] no debug temps for artificial uses
- References: <orobr1e7v2.fsf@livre.localdomain>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Apr 09, 2012 at 03:29:05AM -0300, Alexandre Oliva wrote:
> from Alexandre Oliva <aoliva@redhat.com>
>
> PR debug/50826
> * df-problems.c (df_note_bb_compute): Do not take note of
> debug uses for whose REGs we won't emit DEAD or UNUSED notes.
> Index: gcc/df-problems.c
> ===================================================================
> --- gcc/df-problems.c.orig 2012-04-08 01:50:45.379141149 -0300
> +++ gcc/df-problems.c 2012-04-08 02:05:13.406574358 -0300
> @@ -3528,7 +3528,12 @@ df_note_bb_compute (unsigned int bb_inde
> {
> if (debug_insn > 0)
> {
> - dead_debug_add (&debug, use, uregno);
> + /* We won't add REG_UNUSED or REG_DEAD notes for
> + these, so we don't have to mess with them in
> + debug insns either. */
> + if (!bitmap_bit_p (artificial_uses, uregno)
> + && (!df_ignore_stack_reg (uregno)))
Please remove the extra () around this line,
&& !df_ignore_stack_reg (uregno))
will be more readable.
> + dead_debug_add (&debug, use, uregno);
> continue;
> }
> break;
Ok for trunk with that change, thanks.
Jakub