This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Set current_function_decl in {push,pop}_cfun and push_struct_function
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 9 Aug 2012 16:58:07 +0200
- Subject: Re: [PATCH] Set current_function_decl in {push,pop}_cfun and push_struct_function
- References: <20120809142644.GA11133@virgil.arch.suse.de>
On Thu, Aug 9, 2012 at 4:26 PM, Martin Jambor <mjambor@suse.cz> wrote:
> Hi,
>
> I've always found it silly that in order to change the current
> function one has to call push_cfun and pop_cfun which conveniently set
> and restore the value of cfun and in addition to that also set
> current_function_decl and usually also cache its old value to restore
> it back afterwards. I also think that, at least throughout the
> middle-end, we should strive to have current_function_decl consistent
> with cfun->decl. There are quite a few places where we are not
> consistent and I think such situations are prone to nasty surprises as
> various functions rely on cfun and others on current_function_decl and
> it's easy to be unaware that one of the two is incorrect at the
> moment.
>
> This week I have therefore decided to try and make push_cfun, pop_cfun
> and push_struct_function also set the current_function_decl. Being
> afraid of opening a giant can of worms I only a mid-sized hole and
> left various set_cfuns for later as well as places where we set
> current_function_decl without bothering with cfun. After a few
> debugging sessions I came up with the patch below. The changes are
> mostly mechanical, let me try and explain some of the difficult or
> not-quite-nice ones, most of which come from calls from front-ends
> which generally do not care about cfun all that much.
>
> - In order to ensure that pop_cfun will reliable restore the old
> current_function_decl, push_cfun asserts that cfun and
> current_function_decl match. pop_cfun then simply restores
> current_function_decl to new_cfun->decl or NULL_TREE if new_cfun is
> NULL. To check that the two remain consistent, pop_cfun has a
> similar (albeit checking) assert.
>
> - I had to allow push_cfun(NULL) because in
> gfc_get_extern_function_decl in fortran/trans-decl.c we momentarily
> emulate top-level context by doing:
>
> current_function_decl = NULL_TREE;
> push_cfun (cfun);
>
> do_something ()
>
> pop_cfun ();
> current_function_decl = save_fn_decl;
>
> and to keep current_function_decl consistent with cfun, cfun had to
> be made NULL too. Co I converted the above to push_cfun (NULL)
> which also sets current_function_decl to NULL_TREE.
>
> - I also had to allow push_cfun(NULL) because
> dwarf2out_abstract_function does just that, just it looks like:
>
> push_cfun (DECL_STRUCT_FUNCTION (decl));
>
> but DECL_STRUCT_FUNCTION is usually (always?) NULL for abstract
> origin functions. But this also means that changed push_cfun sets
> current_function_decl to NULL, which means the abstract function is
> not dwarf2outed as it should be. Thus, in perhaps the most awful
> thunk in this patch I re-set current_function_decl after calling
> push_cfun. If someone has a better idea how to deal with this, I'm
> certainly interested.
>
> For the same reason I do not assert that
> current_function matches cfun->decl in pop_cfun if cfun is NULL.
>
> - each cfun change also triggers a pair of init_dummy_function_start
> and expand_dummy_function_end which invoke push_struct_function and
> pop_cfun respectively. Because we may be in the middle of another
> push/pop_cfun, the current_function_decl may not match and so the
> asserts are disabled in these cases, fortunately we can recognize
> them by looking at value of in_dummy_function.
>
> - ada/gcc-interface/utils.c:rest_of_subprog_body_compilation calls
> dump_function which in turns calls dump_function_to_file which calls
> push_cfun. But Ada front end has its idea of the
> current_function_decl and there is no cfun which is an inconsistency
> which makes push_cfun assert fail. I "solved" it by temporarily
> setting current_function_decl to NULL_TREE. It's just dumping and I
> thought that dump_function should be considered middle-end and thus
> middle-end invariants should apply.
>
> The patch passes bootstrap and testing on x86_64-linux (all languages
> + ada + obj-c++) and ia64-linux (c,c++,fortran,objc,obj-c++). There
> is some confusing jitter in the go testing results which I have not
> yet looked at (perhaps compare_tests just can't deal with it, there
> are tests reported both as newly failing and newly working etc...) but
> I thought that I'd send the patch now anyway to get some feedback in
> case I was doing something else wrong (I also do not know whether
> anyone but Ian can modify the go front-end). I have also LTO-built
> Mozilla Firefox with the patch.
>
> Well, what do you think?
Well. We should try to get rid of most push/pop_cfun calls, and the middle-end
should never need to look at current_function_decl ... (in practice we have
tree.c and fold-const.c which has to because its shared between FE and
middle-end).
For example the use in estimate_stack_frame_size. Or the uses in IPA
passes. It would be nice to figure out which parts need access to
cfun/current_function_decl in them (thus, arrange cfun/current_function_decl
to be NULL there).
Other than that, yes - for the middle-end cfun and current_fuction_decl
should be consistent.
Richard.
> Martin
>
>
> 2012-08-08 Martin Jambor <mjambor@suse.cz>
>
> * function.c (push_cfun): Check old current_function_decl matches
> old cfun, set new current_function_decl to the decl of the new
> cfun.
> (push_struct_function): Likewise.
> (pop_cfun): Likewise.
> (allocate_struct_function): Move call to
> invoke_set_current_function_hook to the end of the function.
> * cfgexpand.c (estimated_stack_frame_size): Do not set and restore
> current_function_decl.
> * cgraph.c (cgraph_release_function_body): Likewise.
> * cgraphunit.c (cgraph_process_new_functions): Likewise.
> (cgraph_add_new_function): Likewise.
> (cgraph_analyze_function): Likewise.
> (assemble_thunk): Set cfun to NULL at the end.
> (expand_function): Move call to set_cfun downwards.
> * dwarf2out.c (dwarf2out_abstract_function): Do not restore
> current_function_decl.
> * gimple-low.c (record_vars_into): Only check current_function_decl
> before possibly doing push_cfun.
> * gimplify.c (gimplify_function_tree): Do not set and restore
> current_function_decl.
> * ipa-inline-analysis.c (compute_inline_parameters): Likewise.
> (inline_analyze_function): Likewise.
> * ipa-prop.c (ipa_analyze_node): Likewise.
> * ipa-pure-const.c (analyze_function): Likewise.
> * lto-streamer-in.c (lto_input_function_body): Do not set
> current_function_decl.
> * lto-streamer-out.c (output_function): Do not set and restore
> current_function_decl.
> * matrix-reorg.c (transform_allocation_sites): Likewise, also use
> push/pop_cfun instead of set_cfun.
> (matrix_reorg): Do not set and restore current_function_decl.
> * omp-low.c (finalize_task_copyfn): Likewise.
> (expand_omp_taskreg): Likewise.
> (create_task_copyfn): Likewise, move push_cfun up quite a bit.
> * passes.c (dump_passes): Do not set and restore current_function_decl.
> (do_per_function): Likewise.
> (do_per_function_toporder): Likewise.
> * trans-mem.c (ipa_tm_scan_irr_function): Likewise.
> (ipa_tm_transform_transaction): Likewise.
> (ipa_tm_transform_clone): Likewise.
> (ipa_tm_execute): Likewise.
> * tree-emutls.c (lower_emutls_function_body): Likewise.
> * tree-inline.c (initialize_cfun): Do not call pop_cfun.
> (tree_function_versioning): Do not call push_cfun, do not set and
> restore current_function_decl. Remove assert checking consistency of
> cfun and current_function_decl.
> * tree-profile.c (tree_profiling): Do not set and restore
> current_function_decl.
> * tree-sra.c (convert_callers_for_node): Do not set
> current_function_decl.
> (convert_callers): Do not restore current_function_decl.
> (modify_function): Do not set current_function_decl.
> * tree-ssa-structalias.c (ipa_pta_execute): Do not set and restore
> current_function_decl.
>
> * fortran/trans-decl.c (gfc_get_extern_function_decl): Do not set and
> restore current_function_decl, push NULL cfun.
> (gfc_init_coarray_decl): Do not set and restore current_function_decl.
>
> * go/gofrontend/gogo-tree.cc (write_initialization_function): Do not
> set and restore current_function_decl.
> (write_globals): Likewise.
> (get_tree): Likewise.
>
> * lto/lto.c (lto_materialize_function): Call push_struct_function
> instead of allocate_struct_function and also call pop_cfun.
>
> * ada/gcc-interface/utils.c (rest_of_subprog_body_compilation): Set
> current_function_decl to NULL before calling dump_function and then
> restore it.
>
>
>
> *** /tmp/KDyIva_utils.c Thu Aug 9 15:21:47 2012
> --- gcc/ada/gcc-interface/utils.c Thu Aug 9 13:44:00 2012
> *************** end_subprog_body (tree body)
> *** 2740,2746 ****
> void
> rest_of_subprog_body_compilation (tree subprog_decl)
> {
> ! /* We cannot track the location of errors past this point. */
> error_gnat_node = Empty;
>
> /* If we're only annotating types, don't actually compile this function. */
> --- 2740,2748 ----
> void
> rest_of_subprog_body_compilation (tree subprog_decl)
> {
> ! tree saved_current_function_decl = current_function_decl;
> !
> ! /* We cannot track the location of errors past this point. */
> error_gnat_node = Empty;
>
> /* If we're only annotating types, don't actually compile this function. */
> *************** rest_of_subprog_body_compilation (tree s
> *** 2748,2754 ****
> --- 2750,2758 ----
> return;
>
> /* Dump functions before gimplification. */
> + current_function_decl = NULL_TREE;
> dump_function (TDI_original, subprog_decl);
> + current_function_decl = saved_current_function_decl;
>
> if (!decl_function_context (subprog_decl))
> cgraph_finalize_function (subprog_decl, false);
> *** /tmp/wurZNc_cfgexpand.c Thu Aug 9 15:21:47 2012
> --- gcc/cfgexpand.c Tue Aug 7 13:15:12 2012
> *************** estimated_stack_frame_size (struct cgrap
> *** 1422,1433 ****
> HOST_WIDE_INT size = 0;
> size_t i;
> tree var;
> - tree old_cur_fun_decl = current_function_decl;
> struct function *fn = DECL_STRUCT_FUNCTION (node->symbol.decl);
>
> - current_function_decl = node->symbol.decl;
> push_cfun (fn);
> -
> FOR_EACH_LOCAL_DECL (fn, i, var)
> if (auto_var_in_fn_p (var, fn->decl))
> size += expand_one_var (var, true, false);
> --- 1422,1430 ----
> *************** estimated_stack_frame_size (struct cgrap
> *** 1442,1448 ****
> fini_vars_expansion ();
> }
> pop_cfun ();
> - current_function_decl = old_cur_fun_decl;
> return size;
> }
>
> --- 1439,1444 ----
> *** /tmp/8XER7b_cgraph.c Thu Aug 9 15:21:47 2012
> --- gcc/cgraph.c Tue Aug 7 13:17:24 2012
> *************** cgraph_release_function_body (struct cgr
> *** 1128,1134 ****
> {
> if (DECL_STRUCT_FUNCTION (node->symbol.decl))
> {
> - tree old_decl = current_function_decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> if (cfun->cfg
> && current_loops)
> --- 1128,1133 ----
> *************** cgraph_release_function_body (struct cgr
> *** 1138,1148 ****
> }
> if (cfun->gimple_df)
> {
> - current_function_decl = node->symbol.decl;
> delete_tree_ssa ();
> delete_tree_cfg_annotations ();
> cfun->eh = NULL;
> - current_function_decl = old_decl;
> }
> if (cfun->cfg)
> {
> --- 1137,1145 ----
> *** /tmp/WL9Mge_cgraphunit.c Thu Aug 9 15:21:47 2012
> --- gcc/cgraphunit.c Wed Aug 8 15:54:56 2012
> *************** cgraph_process_new_functions (void)
> *** 313,319 ****
> if (!node->analyzed)
> cgraph_analyze_function (node);
> push_cfun (DECL_STRUCT_FUNCTION (fndecl));
> - current_function_decl = fndecl;
> if ((cgraph_state == CGRAPH_STATE_IPA_SSA
> && !gimple_in_ssa_p (DECL_STRUCT_FUNCTION (fndecl)))
> /* When not optimizing, be sure we run early local passes anyway
> --- 313,318 ----
> *************** cgraph_process_new_functions (void)
> *** 325,331 ****
> free_dominance_info (CDI_POST_DOMINATORS);
> free_dominance_info (CDI_DOMINATORS);
> pop_cfun ();
> - current_function_decl = NULL;
> cgraph_call_function_insertion_hooks (node);
> break;
>
> --- 324,329 ----
> *************** cgraph_add_new_function (tree fndecl, bo
> *** 495,508 ****
> if (!lowered && cgraph_state == CGRAPH_STATE_EXPANSION)
> {
> push_cfun (DECL_STRUCT_FUNCTION (fndecl));
> - current_function_decl = fndecl;
> gimple_register_cfg_hooks ();
> bitmap_obstack_initialize (NULL);
> execute_pass_list (all_lowering_passes);
> execute_pass_list (pass_early_local_passes.pass.sub);
> bitmap_obstack_release (NULL);
> pop_cfun ();
> - current_function_decl = NULL;
>
> lowered = true;
> }
> --- 493,504 ----
> *************** cgraph_add_new_function (tree fndecl, bo
> *** 521,527 ****
> node->lowered = true;
> cgraph_analyze_function (node);
> push_cfun (DECL_STRUCT_FUNCTION (fndecl));
> - current_function_decl = fndecl;
> gimple_register_cfg_hooks ();
> bitmap_obstack_initialize (NULL);
> if (!gimple_in_ssa_p (DECL_STRUCT_FUNCTION (fndecl)))
> --- 517,522 ----
> *************** cgraph_add_new_function (tree fndecl, bo
> *** 529,535 ****
> bitmap_obstack_release (NULL);
> pop_cfun ();
> expand_function (node);
> - current_function_decl = NULL;
> break;
>
> default:
> --- 524,529 ----
> *************** fixup_same_cpp_alias_visibility (symtab_
> *** 597,603 ****
> static void
> cgraph_analyze_function (struct cgraph_node *node)
> {
> - tree save = current_function_decl;
> tree decl = node->symbol.decl;
> location_t saved_loc = input_location;
> input_location = DECL_SOURCE_LOCATION (decl);
> --- 591,596 ----
> *************** cgraph_analyze_function (struct cgraph_n
> *** 638,644 ****
> }
> else
> {
> - current_function_decl = decl;
> push_cfun (DECL_STRUCT_FUNCTION (decl));
>
> assign_assembler_name_if_neeeded (node->symbol.decl);
> --- 631,636 ----
> *************** cgraph_analyze_function (struct cgraph_n
> *** 672,678 ****
> }
> node->analyzed = true;
>
> - current_function_decl = save;
> input_location = saved_loc;
> }
>
> --- 664,669 ----
> *************** assemble_thunk (struct cgraph_node *node
> *** 1520,1525 ****
> --- 1511,1517 ----
> bitmap_obstack_release (NULL);
> }
> current_function_decl = NULL;
> + set_cfun (NULL);
> }
>
>
> *************** expand_function (struct cgraph_node *nod
> *** 1612,1619 ****
> /* Release the default bitmap obstack. */
> bitmap_obstack_release (NULL);
>
> - set_cfun (NULL);
> -
> /* If requested, warn about function definitions where the function will
> return a value (usually of some struct or union type) which itself will
> take up a lot of stack space. */
> --- 1604,1609 ----
> *************** expand_function (struct cgraph_node *nod
> *** 1658,1663 ****
> --- 1648,1654 ----
>
> /* Make sure that BE didn't give up on compiling. */
> gcc_assert (TREE_ASM_WRITTEN (decl));
> + set_cfun (NULL);
> current_function_decl = NULL;
>
> /* It would make a lot more sense to output thunks before function body to get more
> *** /tmp/edrpua_dwarf2out.c Thu Aug 9 15:21:47 2012
> --- gcc/dwarf2out.c Wed Aug 8 18:12:07 2012
> *************** static void
> *** 16673,16679 ****
> dwarf2out_abstract_function (tree decl)
> {
> dw_die_ref old_die;
> - tree save_fn;
> tree context;
> int was_abstract;
> htab_t old_decl_loc_table;
> --- 16673,16678 ----
> *************** dwarf2out_abstract_function (tree decl)
> *** 16714,16736 ****
> }
>
> /* Pretend we've just finished compiling this function. */
> - save_fn = current_function_decl;
> - current_function_decl = decl;
> push_cfun (DECL_STRUCT_FUNCTION (decl));
> !
> was_abstract = DECL_ABSTRACT (decl);
> set_decl_abstract_flags (decl, 1);
> dwarf2out_decl (decl);
> if (! was_abstract)
> set_decl_abstract_flags (decl, 0);
>
> ! current_function_decl = save_fn;
> decl_loc_table = old_decl_loc_table;
> cached_dw_loc_list_table = old_cached_dw_loc_list_table;
> call_arg_locations = old_call_arg_locations;
> call_site_count = old_call_site_count;
> tail_call_site_count = old_tail_call_site_count;
> - pop_cfun ();
> }
>
> /* Helper function of premark_used_types() which gets called through
> --- 16713,16735 ----
> }
>
> /* Pretend we've just finished compiling this function. */
> push_cfun (DECL_STRUCT_FUNCTION (decl));
> ! /* DECL_STRUCT_FUNCTION can be NULL and in that case push_cfun has set
> ! current_function_decl to NULL which is not what we want. Re-set it
> ! again. */
> ! current_function_decl = decl;
> was_abstract = DECL_ABSTRACT (decl);
> set_decl_abstract_flags (decl, 1);
> dwarf2out_decl (decl);
> if (! was_abstract)
> set_decl_abstract_flags (decl, 0);
>
> ! pop_cfun ();
> decl_loc_table = old_decl_loc_table;
> cached_dw_loc_list_table = old_cached_dw_loc_list_table;
> call_arg_locations = old_call_arg_locations;
> call_site_count = old_call_site_count;
> tail_call_site_count = old_tail_call_site_count;
> }
>
> /* Helper function of premark_used_types() which gets called through
> *** /tmp/yrx7vc_trans-decl.c Thu Aug 9 15:21:47 2012
> --- gcc/fortran/trans-decl.c Tue Aug 7 16:02:30 2012
> *************** gfc_get_extern_function_decl (gfc_symbol
> *** 1624,1640 ****
> /* By construction, the external function cannot be
> a contained procedure. */
> locus old_loc;
> - tree save_fn_decl = current_function_decl;
>
> - current_function_decl = NULL_TREE;
> gfc_save_backend_locus (&old_loc);
> ! push_cfun (cfun);
>
> gfc_create_function_decl (gsym->ns, true);
>
> pop_cfun ();
> gfc_restore_backend_locus (&old_loc);
> - current_function_decl = save_fn_decl;
> }
>
> /* If the namespace has entries, the proc_name is the
> --- 1624,1637 ----
> /* By construction, the external function cannot be
> a contained procedure. */
> locus old_loc;
>
> gfc_save_backend_locus (&old_loc);
> ! push_cfun (NULL);
>
> gfc_create_function_decl (gsym->ns, true);
>
> pop_cfun ();
> gfc_restore_backend_locus (&old_loc);
> }
>
> /* If the namespace has entries, the proc_name is the
> *************** add_argument_checking (stmtblock_t *bloc
> *** 4849,4864 ****
> void
> gfc_init_coarray_decl (bool main_tu)
> {
> - tree save_fn_decl;
> -
> if (gfc_option.coarray != GFC_FCOARRAY_LIB)
> return;
>
> if (gfort_gvar_caf_this_image || gfort_gvar_caf_num_images)
> return;
>
> - save_fn_decl = current_function_decl;
> - current_function_decl = NULL_TREE;
> push_cfun (cfun);
>
> gfort_gvar_caf_this_image
> --- 4846,4857 ----
> *************** gfc_init_coarray_decl (bool main_tu)
> *** 4894,4900 ****
> pushdecl_top_level (gfort_gvar_caf_num_images);
>
> pop_cfun ();
> - current_function_decl = save_fn_decl;
> }
>
>
> --- 4887,4892 ----
> *** /tmp/ccrTha_function.c Thu Aug 9 15:21:47 2012
> --- gcc/function.c Wed Aug 8 18:06:40 2012
> *************** set_cfun (struct function *new_cfun)
> *** 4413,4434 ****
>
> static VEC(function_p,heap) *cfun_stack;
>
> ! /* Push the current cfun onto the stack, and set cfun to new_cfun. */
>
> void
> push_cfun (struct function *new_cfun)
> {
> VEC_safe_push (function_p, heap, cfun_stack, cfun);
> set_cfun (new_cfun);
> }
>
> ! /* Pop cfun from the stack. */
>
> void
> pop_cfun (void)
> {
> struct function *new_cfun = VEC_pop (function_p, cfun_stack);
> set_cfun (new_cfun);
> }
>
> /* Return value of funcdef and increase it. */
> --- 4413,4445 ----
>
> static VEC(function_p,heap) *cfun_stack;
>
> ! /* Push the current cfun onto the stack, and set cfun to new_cfun. Also set
> ! current_function_decl accordingly. */
>
> void
> push_cfun (struct function *new_cfun)
> {
> + gcc_assert ((!cfun && !current_function_decl)
> + || (cfun && current_function_decl == cfun->decl));
> VEC_safe_push (function_p, heap, cfun_stack, cfun);
> + current_function_decl = new_cfun ? new_cfun->decl : NULL_TREE;
> set_cfun (new_cfun);
> }
>
> ! /* Pop cfun from the stack. Also set current_function_decl accordingly. */
>
> void
> pop_cfun (void)
> {
> struct function *new_cfun = VEC_pop (function_p, cfun_stack);
> + /* When in_dummy_function, we do have a cfun but current_function_decl is
> + NULL. OTOH, dwarf2out can push a function of an abstract origin decl
> + which is NULL but needs non-null current_function_decl. */
> + gcc_checking_assert (in_dummy_function
> + || !cfun
> + || current_function_decl == cfun->decl);
> set_cfun (new_cfun);
> + current_function_decl = new_cfun ? new_cfun->decl : NULL_TREE;
> }
>
> /* Return value of funcdef and increase it. */
> *************** allocate_struct_function (tree fndecl, b
> *** 4475,4482 ****
> OVERRIDE_ABI_FORMAT (fndecl);
> #endif
>
> - invoke_set_current_function_hook (fndecl);
> -
> if (fndecl != NULL_TREE)
> {
> DECL_STRUCT_FUNCTION (fndecl) = cfun;
> --- 4486,4491 ----
> *************** allocate_struct_function (tree fndecl, b
> *** 4502,4507 ****
> --- 4511,4518 ----
> but is this worth the hassle? */
> cfun->can_throw_non_call_exceptions = flag_non_call_exceptions;
> }
> +
> + invoke_set_current_function_hook (fndecl);
> }
>
> /* This is like allocate_struct_function, but pushes a new cfun for FNDECL
> *************** allocate_struct_function (tree fndecl, b
> *** 4510,4516 ****
> --- 4521,4533 ----
> void
> push_struct_function (tree fndecl)
> {
> + /* When in_dummy_function we might be in the middle of a pop_cfun and
> + current_function_decl and cfun may not match. */
> + gcc_assert (in_dummy_function
> + || (!cfun && !current_function_decl)
> + || (cfun && current_function_decl == cfun->decl));
> VEC_safe_push (function_p, heap, cfun_stack, cfun);
> + current_function_decl = fndecl;
> allocate_struct_function (fndecl, false);
> }
>
> *** /tmp/apOtwc_gimple-low.c Thu Aug 9 15:21:47 2012
> --- gcc/gimple-low.c Tue Aug 7 13:31:20 2012
> *************** lower_builtin_setjmp (gimple_stmt_iterat
> *** 991,997 ****
> void
> record_vars_into (tree vars, tree fn)
> {
> ! if (fn != current_function_decl)
> push_cfun (DECL_STRUCT_FUNCTION (fn));
>
> for (; vars; vars = DECL_CHAIN (vars))
> --- 991,999 ----
> void
> record_vars_into (tree vars, tree fn)
> {
> ! bool change_cfun = fn != current_function_decl;
> !
> ! if (change_cfun)
> push_cfun (DECL_STRUCT_FUNCTION (fn));
>
> for (; vars; vars = DECL_CHAIN (vars))
> *************** record_vars_into (tree vars, tree fn)
> *** 1011,1017 ****
> add_local_decl (cfun, var);
> }
>
> ! if (fn != current_function_decl)
> pop_cfun ();
> }
>
> --- 1013,1019 ----
> add_local_decl (cfun, var);
> }
>
> ! if (change_cfun)
> pop_cfun ();
> }
>
> *** /tmp/uvszca_gimplify.c Thu Aug 9 15:21:47 2012
> --- gcc/gimplify.c Tue Aug 7 13:33:19 2012
> *************** flag_instrument_functions_exclude_p (tre
> *** 8258,8271 ****
> void
> gimplify_function_tree (tree fndecl)
> {
> ! tree oldfn, parm, ret;
> gimple_seq seq;
> gimple bind;
>
> gcc_assert (!gimple_body (fndecl));
>
> - oldfn = current_function_decl;
> - current_function_decl = fndecl;
> if (DECL_STRUCT_FUNCTION (fndecl))
> push_cfun (DECL_STRUCT_FUNCTION (fndecl));
> else
> --- 8258,8269 ----
> void
> gimplify_function_tree (tree fndecl)
> {
> ! tree parm, ret;
> gimple_seq seq;
> gimple bind;
>
> gcc_assert (!gimple_body (fndecl));
>
> if (DECL_STRUCT_FUNCTION (fndecl))
> push_cfun (DECL_STRUCT_FUNCTION (fndecl));
> else
> *************** gimplify_function_tree (tree fndecl)
> *** 8350,8356 ****
> DECL_SAVED_TREE (fndecl) = NULL_TREE;
> cfun->curr_properties = PROP_gimple_any;
>
> - current_function_decl = oldfn;
> pop_cfun ();
> }
>
> --- 8348,8353 ----
> *** /tmp/gISRxc_gogo-tree.cc Thu Aug 9 15:21:47 2012
> --- gcc/go/gofrontend/gogo-tree.cc Tue Aug 7 16:13:51 2012
> *************** Gogo::write_initialization_function(tree
> *** 483,489 ****
>
> DECL_SAVED_TREE(fndecl) = init_stmt_list;
>
> - current_function_decl = fndecl;
> if (DECL_STRUCT_FUNCTION(fndecl) == NULL)
> push_struct_function(fndecl);
> else
> --- 483,488 ----
> *************** Gogo::write_initialization_function(tree
> *** 494,500 ****
>
> cgraph_add_new_function(fndecl, false);
>
> - current_function_decl = NULL_TREE;
> pop_cfun();
> }
>
> --- 493,498 ----
> *************** Gogo::write_globals()
> *** 871,877 ****
> // means that we need an fndecl.
> if (init_fndecl == NULL_TREE)
> init_fndecl = this->initialization_function_decl();
> - current_function_decl = init_fndecl;
> if (DECL_STRUCT_FUNCTION(init_fndecl) == NULL)
> push_struct_function(init_fndecl);
> else
> --- 869,874 ----
> *************** Gogo::write_globals()
> *** 881,887 ****
> var_init_tree = no->var_value()->get_init_block(this, NULL,
> var_decl);
>
> - current_function_decl = NULL_TREE;
> pop_cfun();
> }
>
> --- 878,883 ----
> *************** Named_object::get_tree(Gogo* gogo, Named
> *** 1123,1137 ****
> cfun->function_end_locus =
> func->block()->end_location().gcc_location();
>
> - current_function_decl = decl;
> -
> func->build_tree(gogo, this);
>
> gimplify_function_tree(decl);
>
> cgraph_finalize_function(decl, true);
>
> - current_function_decl = NULL_TREE;
> pop_cfun();
> }
> }
> --- 1119,1130 ----
> *** /tmp/2SvTGa_ipa-inline-analysis.c Thu Aug 9 15:21:47 2012
> --- gcc/ipa-inline-analysis.c Tue Aug 7 13:35:39 2012
> *************** compute_inline_parameters (struct cgraph
> *** 2068,2074 ****
> HOST_WIDE_INT self_stack_size;
> struct cgraph_edge *e;
> struct inline_summary *info;
> - tree old_decl = current_function_decl;
>
> gcc_assert (!node->global.inlined_to);
>
> --- 2068,2073 ----
> *************** compute_inline_parameters (struct cgraph
> *** 2095,2101 ****
> }
>
> /* Even is_gimple_min_invariant rely on current_function_decl. */
> - current_function_decl = node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
>
> /* Estimate the stack size for the function if we're optimizing. */
> --- 2094,2099 ----
> *************** compute_inline_parameters (struct cgraph
> *** 2137,2143 ****
> info->size = info->self_size;
> info->stack_frame_offset = 0;
> info->estimated_stack_size = info->estimated_self_stack_size;
> - current_function_decl = old_decl;
> pop_cfun ();
> }
>
> --- 2135,2140 ----
> *************** static void
> *** 2924,2930 ****
> inline_analyze_function (struct cgraph_node *node)
> {
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> - current_function_decl = node->symbol.decl;
>
> if (dump_file)
> fprintf (dump_file, "\nAnalyzing function: %s/%u\n",
> --- 2921,2926 ----
> *************** inline_analyze_function (struct cgraph_n
> *** 2933,2939 ****
> inline_indirect_intraprocedural_analysis (node);
> compute_inline_parameters (node, false);
>
> - current_function_decl = NULL;
> pop_cfun ();
> }
>
> --- 2929,2934 ----
> *** /tmp/6WwhXc_ipa-prop.c Thu Aug 9 15:21:47 2012
> --- gcc/ipa-prop.c Tue Aug 7 13:36:13 2012
> *************** ipa_analyze_node (struct cgraph_node *no
> *** 1674,1680 ****
> ipa_check_create_edge_args ();
> info = IPA_NODE_REF (node);
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> - current_function_decl = node->symbol.decl;
> ipa_initialize_node_params (node);
>
> param_count = ipa_get_param_count (info);
> --- 1674,1679 ----
> *************** ipa_analyze_node (struct cgraph_node *no
> *** 1688,1694 ****
> if (parms_ainfo[i].visited_statements)
> BITMAP_FREE (parms_ainfo[i].visited_statements);
>
> - current_function_decl = NULL;
> pop_cfun ();
> }
>
> --- 1687,1692 ----
> *** /tmp/QSIQac_ipa-pure-const.c Thu Aug 9 15:21:47 2012
> --- gcc/ipa-pure-const.c Tue Aug 7 13:37:57 2012
> *************** static funct_state
> *** 724,730 ****
> analyze_function (struct cgraph_node *fn, bool ipa)
> {
> tree decl = fn->symbol.decl;
> - tree old_decl = current_function_decl;
> funct_state l;
> basic_block this_block;
>
> --- 724,729 ----
> *************** analyze_function (struct cgraph_node *fn
> *** 752,758 ****
> }
>
> push_cfun (DECL_STRUCT_FUNCTION (decl));
> - current_function_decl = decl;
>
> FOR_EACH_BB (this_block)
> {
> --- 751,756 ----
> *************** end:
> *** 820,826 ****
> l->can_throw = false;
>
> pop_cfun ();
> - current_function_decl = old_decl;
> if (dump_file)
> {
> if (l->looping)
> --- 818,823 ----
> *** /tmp/mNj9je_lto-streamer-in.c Thu Aug 9 15:21:47 2012
> --- gcc/lto-streamer-in.c Tue Aug 7 13:44:16 2012
> *************** void
> *** 1036,1042 ****
> lto_input_function_body (struct lto_file_decl_data *file_data,
> tree fn_decl, const char *data)
> {
> - current_function_decl = fn_decl;
> lto_read_body (file_data, fn_decl, data, LTO_section_function_body);
> }
>
> --- 1036,1041 ----
> *** /tmp/Mmj3jc_lto-streamer-out.c Thu Aug 9 15:21:47 2012
> --- gcc/lto-streamer-out.c Tue Aug 7 13:46:04 2012
> *************** output_function (struct cgraph_node *nod
> *** 793,799 ****
> gcc_assert (current_function_decl == NULL_TREE && cfun == NULL);
>
> /* Set current_function_decl and cfun. */
> - current_function_decl = function;
> push_cfun (fn);
>
> /* Make string 0 be a NULL string. */
> --- 793,798 ----
> *************** output_function (struct cgraph_node *nod
> *** 847,853 ****
>
> destroy_output_block (ob);
>
> - current_function_decl = NULL;
> pop_cfun ();
> }
>
> --- 846,851 ----
> *** /tmp/iXBLWd_lto.c Thu Aug 9 15:21:47 2012
> --- gcc/lto/lto.c Wed Aug 8 12:14:32 2012
> *************** lto_materialize_function (struct cgraph_
> *** 221,227 ****
>
> gcc_assert (DECL_STRUCT_FUNCTION (decl) == NULL);
>
> ! allocate_struct_function (decl, false);
> announce_function (decl);
> lto_input_function_body (file_data, decl, data);
> if (DECL_FUNCTION_PERSONALITY (decl) && !first_personality_decl)
> --- 221,227 ----
>
> gcc_assert (DECL_STRUCT_FUNCTION (decl) == NULL);
>
> ! push_struct_function (decl);
> announce_function (decl);
> lto_input_function_body (file_data, decl, data);
> if (DECL_FUNCTION_PERSONALITY (decl) && !first_personality_decl)
> *************** lto_materialize_function (struct cgraph_
> *** 229,234 ****
> --- 229,235 ----
> lto_stats.num_function_bodies++;
> lto_free_section_data (file_data, LTO_section_function_body, name,
> data, len);
> + pop_cfun ();
> ggc_collect ();
> }
> }
> *** /tmp/mJnAqb_matrix-reorg.c Thu Aug 9 15:21:47 2012
> --- gcc/matrix-reorg.c Tue Aug 7 13:50:23 2012
> *************** transform_allocation_sites (void **slot,
> *** 1985,1991 ****
> {
> int i;
> struct matrix_info *mi;
> ! tree type, oldfn, prev_dim_size;
> gimple call_stmt_0, use_stmt;
> struct cgraph_node *c_node;
> struct cgraph_edge *e;
> --- 1985,1991 ----
> {
> int i;
> struct matrix_info *mi;
> ! tree type, prev_dim_size;
> gimple call_stmt_0, use_stmt;
> struct cgraph_node *c_node;
> struct cgraph_edge *e;
> *************** transform_allocation_sites (void **slot,
> *** 2102,2109 ****
> We add the assignment to the global before the malloc of level 0. */
>
> /* To be able to produce gimple temporaries. */
> - oldfn = current_function_decl;
> - current_function_decl = mi->allocation_function_decl;
> push_cfun (DECL_STRUCT_FUNCTION (mi->allocation_function_decl));
>
> /* Set the dimension sizes as follows:
> --- 2102,2107 ----
> *************** transform_allocation_sites (void **slot,
> *** 2213,2225 ****
> e = cgraph_edge (c_node, mi->free_stmts[i].stmt);
> gcc_assert (e);
> cgraph_remove_edge (e);
> ! current_function_decl = mi->free_stmts[i].func;
> ! set_cfun (DECL_STRUCT_FUNCTION (mi->free_stmts[i].func));
> gsi = gsi_for_stmt (mi->free_stmts[i].stmt);
> gsi_remove (&gsi, true);
> }
> /* Return to the previous situation. */
> - current_function_decl = oldfn;
> pop_cfun ();
> return 1;
>
> --- 2211,2222 ----
> e = cgraph_edge (c_node, mi->free_stmts[i].stmt);
> gcc_assert (e);
> cgraph_remove_edge (e);
> ! push_cfun (DECL_STRUCT_FUNCTION (mi->free_stmts[i].func));
> gsi = gsi_for_stmt (mi->free_stmts[i].stmt);
> gsi_remove (&gsi, true);
> + pop_cfun ();
> }
> /* Return to the previous situation. */
> pop_cfun ();
> return 1;
>
> *************** matrix_reorg (void)
> *** 2265,2274 ****
> /* Analyze the accesses of the matrices (escaping analysis). */
> FOR_EACH_DEFINED_FUNCTION (node)
> {
> - tree temp_fn;
> -
> - temp_fn = current_function_decl;
> - current_function_decl = node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> bitmap_obstack_initialize (NULL);
> gimple_register_cfg_hooks ();
> --- 2262,2267 ----
> *************** matrix_reorg (void)
> *** 2278,2284 ****
> free_dominance_info (CDI_DOMINATORS);
> free_dominance_info (CDI_POST_DOMINATORS);
> pop_cfun ();
> - current_function_decl = temp_fn;
> bitmap_obstack_release (NULL);
>
> return 0;
> --- 2271,2276 ----
> *************** matrix_reorg (void)
> *** 2293,2299 ****
> free_dominance_info (CDI_DOMINATORS);
> free_dominance_info (CDI_POST_DOMINATORS);
> pop_cfun ();
> - current_function_decl = temp_fn;
> bitmap_obstack_release (NULL);
>
> return 0;
> --- 2285,2290 ----
> *************** matrix_reorg (void)
> *** 2324,2330 ****
> free_dominance_info (CDI_DOMINATORS);
> free_dominance_info (CDI_POST_DOMINATORS);
> pop_cfun ();
> - current_function_decl = temp_fn;
> bitmap_obstack_release (NULL);
> }
> htab_traverse (matrices_to_reorg, transform_allocation_sites, NULL);
> --- 2315,2320 ----
> *************** matrix_reorg (void)
> *** 2332,2341 ****
> FOR_EACH_DEFINED_FUNCTION (node)
> {
> /* Remember that allocation sites have been handled. */
> - tree temp_fn;
> -
> - temp_fn = current_function_decl;
> - current_function_decl = node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> bitmap_obstack_initialize (NULL);
> gimple_register_cfg_hooks ();
> --- 2322,2327 ----
> *************** matrix_reorg (void)
> *** 2345,2351 ****
> free_dominance_info (CDI_DOMINATORS);
> free_dominance_info (CDI_POST_DOMINATORS);
> pop_cfun ();
> - current_function_decl = temp_fn;
> bitmap_obstack_release (NULL);
> }
> htab_traverse (matrices_to_reorg, dump_matrix_reorg_analysis, NULL);
> --- 2331,2336 ----
> *** /tmp/EQo3Md_omp-low.c Thu Aug 9 15:21:47 2012
> --- gcc/omp-low.c Tue Aug 7 13:54:21 2012
> *************** static void
> *** 1243,1249 ****
> finalize_task_copyfn (gimple task_stmt)
> {
> struct function *child_cfun;
> ! tree child_fn, old_fn;
> gimple_seq seq = NULL, new_seq;
> gimple bind;
>
> --- 1243,1249 ----
> finalize_task_copyfn (gimple task_stmt)
> {
> struct function *child_cfun;
> ! tree child_fn;
> gimple_seq seq = NULL, new_seq;
> gimple bind;
>
> *************** finalize_task_copyfn (gimple task_stmt)
> *** 1257,1265 ****
> DECL_STRUCT_FUNCTION (child_fn)->curr_properties
> = cfun->curr_properties & ~PROP_loops;
>
> - old_fn = current_function_decl;
> push_cfun (child_cfun);
> - current_function_decl = child_fn;
> bind = gimplify_body (child_fn, false);
> gimple_seq_add_stmt (&seq, bind);
> new_seq = maybe_catch_exception (seq);
> --- 1257,1263 ----
> *************** finalize_task_copyfn (gimple task_stmt)
> *** 1271,1277 ****
> }
> gimple_set_body (child_fn, seq);
> pop_cfun ();
> - current_function_decl = old_fn;
>
> cgraph_add_new_function (child_fn, false);
> }
> --- 1269,1274 ----
> *************** expand_omp_taskreg (struct omp_region *r
> *** 3389,3395 ****
> basic_block entry_bb, exit_bb, new_bb;
> struct function *child_cfun;
> tree child_fn, block, t;
> - tree save_current;
> gimple_stmt_iterator gsi;
> gimple entry_stmt, stmt;
> edge e;
> --- 3386,3391 ----
> *************** expand_omp_taskreg (struct omp_region *r
> *** 3589,3596 ****
> /* Fix the callgraph edges for child_cfun. Those for cfun will be
> fixed in a following pass. */
> push_cfun (child_cfun);
> - save_current = current_function_decl;
> - current_function_decl = child_fn;
> if (optimize)
> optimize_omp_library_calls (entry_stmt);
> rebuild_cgraph_edges ();
> --- 3585,3590 ----
> *************** expand_omp_taskreg (struct omp_region *r
> *** 3611,3617 ****
> }
> if (gimple_in_ssa_p (cfun))
> update_ssa (TODO_update_ssa);
> - current_function_decl = save_current;
> pop_cfun ();
> }
>
> --- 3605,3610 ----
> *************** create_task_copyfn (gimple task_stmt, om
> *** 6460,6466 ****
>
> /* Populate the function. */
> push_gimplify_context (&gctx);
> ! current_function_decl = child_fn;
>
> bind = build3 (BIND_EXPR, void_type_node, NULL, NULL, NULL);
> TREE_SIDE_EFFECTS (bind) = 1;
> --- 6453,6459 ----
>
> /* Populate the function. */
> push_gimplify_context (&gctx);
> ! push_cfun (child_cfun);
>
> bind = build3 (BIND_EXPR, void_type_node, NULL, NULL, NULL);
> TREE_SIDE_EFFECTS (bind) = 1;
> *************** create_task_copyfn (gimple task_stmt, om
> *** 6507,6514 ****
> else
> tcctx.cb.decl_map = NULL;
>
> - push_cfun (child_cfun);
> -
> arg = DECL_ARGUMENTS (child_fn);
> TREE_TYPE (arg) = build_pointer_type (record_type);
> sarg = DECL_CHAIN (arg);
> --- 6500,6505 ----
> *************** create_task_copyfn (gimple task_stmt, om
> *** 6666,6672 ****
> pop_gimplify_context (NULL);
> BIND_EXPR_BODY (bind) = list;
> pop_cfun ();
> ! current_function_decl = ctx->cb.src_fn;
> }
>
> /* Lower the OpenMP parallel or task directive in the current statement
> --- 6657,6664 ----
> pop_gimplify_context (NULL);
> BIND_EXPR_BODY (bind) = list;
> pop_cfun ();
> ! /* !!! Remove after testing. */
> ! gcc_assert (current_function_decl == ctx->cb.src_fn);
> }
>
> /* Lower the OpenMP parallel or task directive in the current statement
> *** /tmp/qpEqgc_passes.c Thu Aug 9 15:21:47 2012
> --- gcc/passes.c Tue Aug 7 13:56:40 2012
> *************** void
> *** 679,685 ****
> dump_passes (void)
> {
> struct cgraph_node *n, *node = NULL;
> - tree save_fndecl = current_function_decl;
>
> create_pass_tab();
>
> --- 679,684 ----
> *************** dump_passes (void)
> *** 694,700 ****
> return;
>
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> - current_function_decl = node->symbol.decl;
>
> dump_pass_list (all_lowering_passes, 1);
> dump_pass_list (all_small_ipa_passes, 1);
> --- 693,698 ----
> *************** dump_passes (void)
> *** 704,710 ****
> dump_pass_list (all_passes, 1);
>
> pop_cfun ();
> - current_function_decl = save_fndecl;
> }
>
>
> --- 702,707 ----
> *************** do_per_function (void (*callback) (void
> *** 1651,1664 ****
> && (!node->clone_of || node->symbol.decl != node->clone_of->symbol.decl))
> {
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> - current_function_decl = node->symbol.decl;
> callback (data);
> if (!flag_wpa)
> {
> free_dominance_info (CDI_DOMINATORS);
> free_dominance_info (CDI_POST_DOMINATORS);
> }
> - current_function_decl = NULL;
> pop_cfun ();
> ggc_collect ();
> }
> --- 1648,1659 ----
> *************** do_per_function_toporder (void (*callbac
> *** 1699,1709 ****
> if (cgraph_function_with_gimple_body_p (node))
> {
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> - current_function_decl = node->symbol.decl;
> callback (data);
> free_dominance_info (CDI_DOMINATORS);
> free_dominance_info (CDI_POST_DOMINATORS);
> - current_function_decl = NULL;
> pop_cfun ();
> ggc_collect ();
> }
> --- 1694,1702 ----
> *** /tmp/GR8iVd_trans-mem.c Thu Aug 9 15:21:47 2012
> --- gcc/trans-mem.c Tue Aug 7 13:59:14 2012
> *************** ipa_tm_scan_irr_function (struct cgraph_
> *** 3980,3986 ****
> || DECL_STRUCT_FUNCTION (node->symbol.decl)->cfg == NULL)
> return false;
>
> - current_function_decl = node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> calculate_dominance_info (CDI_DOMINATORS);
>
> --- 3980,3985 ----
> *************** ipa_tm_scan_irr_function (struct cgraph_
> *** 4052,4058 ****
>
> VEC_free (basic_block, heap, queue);
> pop_cfun ();
> - current_function_decl = NULL;
>
> return ret;
> }
> --- 4051,4056 ----
> *************** ipa_tm_transform_transaction (struct cgr
> *** 4690,4696 ****
>
> d = get_cg_data (&node, true);
>
> - current_function_decl = node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> calculate_dominance_info (CDI_DOMINATORS);
>
> --- 4688,4693 ----
> *************** ipa_tm_transform_transaction (struct cgr
> *** 4715,4721 ****
> update_ssa (TODO_update_ssa_only_virtuals);
>
> pop_cfun ();
> - current_function_decl = NULL;
> }
>
> /* Transform the calls within the transactional clone of NODE. */
> --- 4712,4717 ----
> *************** ipa_tm_transform_clone (struct cgraph_no
> *** 4734,4741 ****
> if (!node->callees && !node->indirect_calls && !d->irrevocable_blocks_clone)
> return;
>
> ! current_function_decl = d->clone->symbol.decl;
> ! push_cfun (DECL_STRUCT_FUNCTION (current_function_decl));
> calculate_dominance_info (CDI_DOMINATORS);
>
> need_ssa_rename =
> --- 4730,4736 ----
> if (!node->callees && !node->indirect_calls && !d->irrevocable_blocks_clone)
> return;
>
> ! push_cfun (DECL_STRUCT_FUNCTION (d->clone->symbol.decl));
> calculate_dominance_info (CDI_DOMINATORS);
>
> need_ssa_rename =
> *************** ipa_tm_transform_clone (struct cgraph_no
> *** 4746,4752 ****
> update_ssa (TODO_update_ssa_only_virtuals);
>
> pop_cfun ();
> - current_function_decl = NULL;
> }
>
> /* Main entry point for the transactional memory IPA pass. */
> --- 4741,4746 ----
> *************** ipa_tm_execute (void)
> *** 4793,4799 ****
> continue;
> }
>
> - current_function_decl = node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> calculate_dominance_info (CDI_DOMINATORS);
>
> --- 4787,4792 ----
> *************** ipa_tm_execute (void)
> *** 4813,4819 ****
> }
>
> pop_cfun ();
> - current_function_decl = NULL;
> }
>
> /* For every local function on the callee list, scan as if we will be
> --- 4806,4811 ----
> *** /tmp/wwqagc_tree-emutls.c Thu Aug 9 15:21:47 2012
> --- gcc/tree-emutls.c Tue Aug 7 14:48:01 2012
> *************** lower_emutls_function_body (struct cgrap
> *** 618,624 ****
> struct lower_emutls_data d;
> bool any_edge_inserts = false;
>
> - current_function_decl = node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
>
> d.cfun_node = node;
> --- 618,623 ----
> *************** lower_emutls_function_body (struct cgrap
> *** 689,695 ****
> gsi_commit_edge_inserts ();
>
> pop_cfun ();
> - current_function_decl = NULL;
> }
>
> /* Create emutls variable for VAR, DATA is pointer to static
> --- 688,693 ----
> *** /tmp/C7R3Td_tree-inline.c Thu Aug 9 15:21:47 2012
> --- gcc/tree-inline.c Tue Aug 7 14:56:44 2012
> *************** remap_decl_1 (tree decl, void *data)
> *** 1998,2004 ****
> }
>
> /* Build struct function and associated datastructures for the new clone
> ! NEW_FNDECL to be build. CALLEE_FNDECL is the original */
>
> static void
> initialize_cfun (tree new_fndecl, tree callee_fndecl, gcov_type count)
> --- 1998,2005 ----
> }
>
> /* Build struct function and associated datastructures for the new clone
> ! NEW_FNDECL to be build. CALLEE_FNDECL is the original. Function changes
> ! the cfun to the function of new_fndecl (and current_function_decl too). */
>
> static void
> initialize_cfun (tree new_fndecl, tree callee_fndecl, gcov_type count)
> *************** initialize_cfun (tree new_fndecl, tree c
> *** 2063,2069 ****
> cfun->gimple_df->in_ssa_p = true;
> init_ssa_operands (cfun);
> }
> - pop_cfun ();
> }
>
> /* Helper function for copy_cfg_body. Move debug stmts from the end
> --- 2064,2069 ----
> *************** tree_function_versioning (tree old_decl,
> *** 5002,5009 ****
> struct ipa_replace_map *replace_info;
> basic_block old_entry_block, bb;
> VEC (gimple, heap) *init_stmts = VEC_alloc (gimple, heap, 10);
> -
> - tree old_current_function_decl = current_function_decl;
> tree vars = NULL_TREE;
>
> gcc_assert (TREE_CODE (old_decl) == FUNCTION_DECL
> --- 5002,5007 ----
> *************** tree_function_versioning (tree old_decl,
> *** 5073,5086 ****
> id.transform_return_to_modify = false;
> id.transform_lang_insert_block = NULL;
>
> - current_function_decl = new_decl;
> old_entry_block = ENTRY_BLOCK_PTR_FOR_FUNCTION
> (DECL_STRUCT_FUNCTION (old_decl));
> initialize_cfun (new_decl, old_decl,
> old_entry_block->count);
> DECL_STRUCT_FUNCTION (new_decl)->gimple_df->ipa_pta
> = id.src_cfun->gimple_df->ipa_pta;
> - push_cfun (DECL_STRUCT_FUNCTION (new_decl));
>
> /* Copy the function's static chain. */
> p = DECL_STRUCT_FUNCTION (old_decl)->static_chain_decl;
> --- 5071,5082 ----
> *************** tree_function_versioning (tree old_decl,
> *** 5234,5242 ****
> gcc_assert (!id.debug_stmts);
> VEC_free (gimple, heap, init_stmts);
> pop_cfun ();
> - current_function_decl = old_current_function_decl;
> - gcc_assert (!current_function_decl
> - || DECL_STRUCT_FUNCTION (current_function_decl) == cfun);
> return;
> }
>
> --- 5230,5235 ----
> *** /tmp/QSouyb_tree-profile.c Thu Aug 9 15:21:47 2012
> --- gcc/tree-profile.c Tue Aug 7 14:59:02 2012
> *************** tree_profiling (void)
> *** 479,485 ****
> continue;
>
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> - current_function_decl = node->symbol.decl;
>
> /* Re-set global shared temporary variable for edge-counters. */
> gcov_type_tmp_var = NULL_TREE;
> --- 479,484 ----
> *************** tree_profiling (void)
> *** 504,511 ****
> easy to adjust it, if and when there is some. */
> free_dominance_info (CDI_DOMINATORS);
> free_dominance_info (CDI_POST_DOMINATORS);
> -
> - current_function_decl = NULL;
> pop_cfun ();
> }
>
> --- 503,508 ----
> *************** tree_profiling (void)
> *** 540,546 ****
> continue;
>
> push_cfun (DECL_STRUCT_FUNCTION (node->symbol.decl));
> - current_function_decl = node->symbol.decl;
>
> FOR_EACH_BB (bb)
> {
> --- 537,542 ----
> *************** tree_profiling (void)
> *** 557,563 ****
>
> rebuild_cgraph_edges ();
>
> - current_function_decl = NULL;
> pop_cfun ();
> }
>
> --- 553,558 ----
> *** /tmp/2E21Od_tree-sra.c Thu Aug 9 15:21:47 2012
> --- gcc/tree-sra.c Tue Aug 7 15:01:55 2012
> *************** convert_callers_for_node (struct cgraph_
> *** 4612,4618 ****
>
> for (cs = node->callers; cs; cs = cs->next_caller)
> {
> - current_function_decl = cs->caller->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (cs->caller->symbol.decl));
>
> if (dump_file)
> --- 4612,4617 ----
> *************** static void
> *** 4641,4654 ****
> convert_callers (struct cgraph_node *node, tree old_decl,
> ipa_parm_adjustment_vec adjustments)
> {
> - tree old_cur_fndecl = current_function_decl;
> basic_block this_block;
>
> cgraph_for_node_and_aliases (node, convert_callers_for_node,
> adjustments, false);
>
> - current_function_decl = old_cur_fndecl;
> -
> if (!encountered_recursive_call)
> return;
>
> --- 4640,4650 ----
> *************** modify_function (struct cgraph_node *nod
> *** 4689,4699 ****
> rebuild_cgraph_edges ();
> free_dominance_info (CDI_DOMINATORS);
> pop_cfun ();
> - current_function_decl = NULL_TREE;
>
> new_node = cgraph_function_versioning (node, redirect_callers, NULL, NULL,
> false, NULL, NULL, "isra");
> - current_function_decl = new_node->symbol.decl;
> push_cfun (DECL_STRUCT_FUNCTION (new_node->symbol.decl));
>
> ipa_modify_formal_parameters (current_function_decl, adjustments, "ISRA");
> --- 4685,4693 ----
> *** /tmp/60W3rb_tree-ssa-structalias.c Thu Aug 9 15:21:47 2012
> --- gcc/tree-ssa-structalias.c Tue Aug 7 15:42:56 2012
> *************** ipa_pta_execute (void)
> *** 6891,6897 ****
> {
> struct function *func;
> basic_block bb;
> - tree old_func_decl;
>
> /* Nodes without a body are not interesting. */
> if (!cgraph_function_with_gimple_body_p (node))
> --- 6891,6896 ----
> *************** ipa_pta_execute (void)
> *** 6909,6917 ****
> }
>
> func = DECL_STRUCT_FUNCTION (node->symbol.decl);
> - old_func_decl = current_function_decl;
> push_cfun (func);
> - current_function_decl = node->symbol.decl;
>
> /* For externally visible or attribute used annotated functions use
> local constraints for their arguments.
> --- 6908,6914 ----
> *************** ipa_pta_execute (void)
> *** 6968,6974 ****
> }
> }
>
> - current_function_decl = old_func_decl;
> pop_cfun ();
>
> if (dump_file)
> --- 6965,6970 ----