> 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?

Honza

Reply via email to