Re: [PATCH 4/7] Break up determine_known_aggregate_parts

[Martin Jambor <mjambor@suse.cz>]

On Mon, May 26, 2014 at 02:54:17AM +0200, Jan Hubicka wrote:
> > Hi,
> > 
> > the main purpose of this patch is to break up function
> > determine_known_aggregate_parts so that the next patch can use the
> > standalone bits and to make the changes slightly easier for review.
> > 
> > However, this patch also removes some of the offset checks which Honza
> > correctly thought superfluous and even possibly filtering out useful
> > information.
> >  
> > Bootstrapped and tested and LTO-bootstrapped on x86_64-linux.
> > OK for trunk after the preceeding patches get in?
> > 
> > Thanks,
> > 
> > Martin
> > 
> > 
> > 2014-02-19  Martin Jambor  <mjambor@suse.cz>
> > 
> > 	* ipa-prop.c (get_place_in_agg_contents_list): New function.
> > 	(build_agg_jump_func_from_list): Likewise.
> > 	(determine_known_aggregate_parts): Renamed to
> > 	determine_locally_known_aggregate_parts.  Moved some functionality
> > 	to the two functions above, removed bound checks.
> 
> This is OK.
> Does it depend on part 4?
> 

No, it does not.  I have re-tested the new order of the patches and
committed this one.

Thanks,

Martin