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 <mjam...@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
