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] Another canonical cselib_val bootstrap fix (PR bootstrap/51725)

From: Jakub Jelinek <jakub at redhat dot com>
To: Richard Guenther <rguenther at suse dot de>
Cc: Richard Henderson <rth at redhat dot com>, Alexandre Oliva <aoliva at redhat dot com>, gcc-patches at gcc dot gnu dot org
Date: Wed, 4 Jan 2012 10:43:53 +0100
Subject: Re: [PATCH] Another canonical cselib_val bootstrap fix (PR bootstrap/51725)
References: <20120103191457.GZ18937@tyan-ft48-01.lab.bos.redhat.com> <alpine.LNX.2.00.1201041033160.4999@zhemvz.fhfr.qr>
Reply-to: Jakub Jelinek <jakub at redhat dot com>

On Wed, Jan 04, 2012 at 10:33:50AM +0100, Richard Guenther wrote:
> > My previous patch apparently wasn't enough.  If at add_mem_*
> > time mem_elt is still its own canonical cselib_val, but only afterwards
> > new_elt_loc_list adds a new canonical cselib_val to it (and moves
> 
> Hm, shouldn't an existing cselib_val be always the canonical one?

Canonical is the one with lower uid.  If cselib_add_permanent_equiv
is called with two existing cselib_vals, just one of them will be
canonical.

	Jakub

References:
- [PATCH] Another canonical cselib_val bootstrap fix (PR bootstrap/51725)
  - From: Jakub Jelinek
- Re: [PATCH] Another canonical cselib_val bootstrap fix (PR bootstrap/51725)
  - From: Richard Guenther

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