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] Fix dead_debug_insert_before ICE (PR debug/49522, take 2)

From: Eric Botcazou <ebotcazou at adacore dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Wed, 6 Jul 2011 22:36:02 +0200
Subject: Re: [PATCH] Fix dead_debug_insert_before ICE (PR debug/49522, take 2)
References: <20110704172938.GT16443@tyan-ft48-01.lab.bos.redhat.com> <20110705200651.GG2687@tyan-ft48-01.lab.bos.redhat.com> <20110706194252.GJ2687@tyan-ft48-01.lab.bos.redhat.com>

> And here is a version that passed bootstrap/regtest on x86_64-linux and
> i686-linux:
>
> 2011-07-06  Jakub Jelinek  <jakub@redhat.com>
>
> 	PR debug/49522
> 	* df-problems.c (dead_debug_reset): Remove dead_debug_uses
> 	referencing debug insns that have been reset.
> 	(dead_debug_insert_before): Don't assert reg is non-NULL,
> 	instead return immediately if it is NULL.
>
> 	* gcc.dg/debug/pr49522.c: New test.

Sorry, our messages crossed.  I'd set a flag in the first loop.  In the end, 
it's up to you.

-- 
Eric Botcazou

Follow-Ups:
- [PATCH] Fix dead_debug_insert_before ICE (PR debug/49522, take 3)
  - From: Jakub Jelinek

References:
- [PATCH] Fix dead_debug_insert_before ICE (PR debug/49522)
  - From: Jakub Jelinek
- Re: [PATCH] Fix dead_debug_insert_before ICE (PR debug/49522)
  - From: Jakub Jelinek
- [PATCH] Fix dead_debug_insert_before ICE (PR debug/49522, take 2)
  - From: Jakub Jelinek

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