On Fri, Sep 13, 2013 at 03:15:56PM +0200, Jakub Jelinek wrote: > On Fri, Sep 13, 2013 at 05:11:09PM +0400, Michael V. Zolotukhin wrote: > > > FYI, I'm attaching a WIP patch with the splay tree stuff. > > Thanks, I'll take a look. By the way, isn't it better to move splay-tree > > implementation to a separate file? > > As it is just a few routines, heavily modified from include/splay-tree.h > (e.g. the data structures contain all the target.c specific stuff), and will > be > used just in target.c, I think it is fine to keep it in target.c.
Anyway, here is an updated patch that moves the splay stuff into splay-tree.h and cleans up a bunch of other things. Will commit once the http://gcc.gnu.org/ml/gcc-patches/2013-09/msg01044.html issue is resolved. 2013-09-13 Jakub Jelinek <ja...@redhat.com> * ipa-prop.c (ipa_compute_jump_functions_for_edge): Return early for internal calls. --- gcc/ipa-prop.c.jj 2013-09-13 16:48:54.000000000 +0200 +++ gcc/ipa-prop.c 2013-09-13 17:28:28.086058903 +0200 @@ -1551,6 +1551,8 @@ ipa_compute_jump_functions_for_edge (str return; vec_safe_grow_cleared (args->jump_functions, arg_num); + if (gimple_call_internal_p (call)) + return; if (ipa_func_spec_opts_forbid_analysis_p (cs->caller)) return; Jakub
