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] O(1) PHI argument look-up - Part 1/n

From: Diego Novillo <dnovillo at redhat dot com>
To: Kazu Hirata <kazu at cs dot umass dot edu>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Fri, 19 Nov 2004 16:39:11 -0500
Subject: Re: [patch] O(1) PHI argument look-up - Part 1/n
Organization: Red Hat Canada
References: <20041118.095455.74724221.kazu@cs.umass.edu> <1100877983.13864.14.camel@localhost.localdomain> <20041119.163507.74729604.kazu@cs.umass.edu>

On Fri, 2004-11-19 at 16:35 -0500, Kazu Hirata wrote:

> Otherwise, VEC_unordered_remove fills a "hole" with the last
> element in the edge vector, at which point, the edge that now lives in
> the "hole" no longer has a consistent dest_idx, so I update its
> dest_idx.  Maybe I should have put this comment there.
> 
Ah, you are right.  Sorry about that.  Yes, a comment should help
clarify things.


> Can I go with the original patch except that I address the comment
> issues above?
> 
Sure.


Diego.

Follow-Ups:
- Re: [patch] O(1) PHI argument look-up - Part 1/n
  - From: Kazu Hirata

References:
- [patch] O(1) PHI argument look-up - Part 1/n
  - From: Kazu Hirata
- Re: [patch] O(1) PHI argument look-up - Part 1/n
  - From: Diego Novillo
- Re: [patch] O(1) PHI argument look-up - Part 1/n
  - From: Kazu Hirata

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