On Fri, Sep 2, 2011 at 8:24 AM, Martin Jambor <mjam...@suse.cz> wrote: > Hi, > > when I submitted the new IPA-CP a few months ago Honza requested that > I store the jump_functions in a VECtor rather than an array in which > they are now. The patch below does exactly that. > > The last remaining such request is to rename > ipa_check_create_node_params to "something else." I guess I'll leave > this to the next weekend when we'll see each other. > > I hope I got the ggc stuff right and a GTY marker does not need to be > added somewhere. On the other hand, the patch passes bootstrap and > testsuite on x86_64-linux and I have successfully LTO-built Firefox > with it (which was able to display some complicated pages). > > OK for trunk? > > Thanks, > > Martin > > > 2011-09-02 Martin Jambor <mjam...@suse.cz> > > * ipa-prop.h (ipa_jump_func_t): New typedef. > (struct ipa_edge_args): Removed field argument_count, field > jump_functions turned into a vector. > (ipa_set_cs_argument_count): Removed. > (ipa_get_cs_argument_count): Updated to work on vectors. > (ipa_get_ith_jump_func): Likewise. > * ipa-prop.c (ipa_count_arguments): Removed. > (compute_scalar_jump_functions): Use ipa_get_ith_jump_func to access > jump functions. Update caller. > (compute_pass_through_member_ptrs): Likewise. > (compute_cst_member_ptr_arguments): Likewise. > (ipa_compute_jump_functions_for_edge): Get number of arguments from > the statement, allocate vector. > (ipa_compute_jump_functions): Do not call ipa_count_arguments. > (duplicate_ipa_jump_func_array): Removed. > (ipa_edge_duplication_hook): Use VEC_copy, do not copy argument count. > (ipa_read_node_info): Allocate vector. > >
This caused: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50301 -- H.J.
