Re: gphi_iterator? (was Re: [PATCH 09/89] Introduce gimple_phi_iterator)

2014-05-14 Thread Jeff Law
On 05/14/14 09:28, David Malcolm wrote: Thanks. I'm working my way through the patches, converting them to the latest approach, where "gimple_phi" becomes "gphi *". Given that renaming, it occurred to me that the new class "gimple_phi_iterator" would be better named as "gphi_iterator" (consist

gphi_iterator? (was Re: [PATCH 09/89] Introduce gimple_phi_iterator)

2014-05-14 Thread David Malcolm
On Fri, 2014-05-09 at 12:51 -0600, Jeff Law wrote: > On 04/21/14 10:56, David Malcolm wrote: > > gcc/ > > * gimple-iterator.h (struct gimple_phi_iterator): New subclass of > > gimple_stmt_iterator with identical layout, but adding... > > (gimple_phi_iterator::phi): ...new method, equiva

Re: [PATCH 09/89] Introduce gimple_phi_iterator

2014-05-09 Thread Jeff Law
On 04/21/14 10:56, David Malcolm wrote: gcc/ * gimple-iterator.h (struct gimple_phi_iterator): New subclass of gimple_stmt_iterator with identical layout, but adding... (gimple_phi_iterator::phi): ...new method, equivalent to gsi_stmt (), but casting the underlying