From c12ee5df35dad4a053195d2e8aef16b7758d77d8 Mon Sep 17 00:00:00 2001 From: Mikael Morin Date: Wed, 6 Oct 2010 16:05:41 +0000 Subject: [PATCH] trans-expr.c (get_proc_ptr_comp): Restore initial expression type before calling gfc_free_expr. 2010-10-06 Mikael Morin * trans-expr.c (get_proc_ptr_comp): Restore initial expression type before calling gfc_free_expr. From-SVN: r165047 --- gcc/fortran/ChangeLog | 5 +++++ gcc/fortran/trans-expr.c | 7 +++++++ 2 files changed, 12 insertions(+) diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 4c8cec42321..dc0bfbb1e38 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,8 @@ +2010-10-06 Mikael Morin + + * trans-expr.c (get_proc_ptr_comp): Restore initial expression type + before calling gfc_free_expr. + 2010-10-06 Mikael Morin * trans-array.c (gfc_conv_tmp_array_ref): Add factorized call to diff --git a/gcc/fortran/trans-expr.c b/gcc/fortran/trans-expr.c index d662d200208..c1588f8c749 100644 --- a/gcc/fortran/trans-expr.c +++ b/gcc/fortran/trans-expr.c @@ -1612,10 +1612,17 @@ get_proc_ptr_comp (gfc_expr *e) { gfc_se comp_se; gfc_expr *e2; + expr_t old_type; + gfc_init_se (&comp_se, NULL); e2 = gfc_copy_expr (e); + /* We have to restore the expr type later so that gfc_free_expr frees + the exact same thing that was allocated. + TODO: This is ugly. */ + old_type = e2->expr_type; e2->expr_type = EXPR_VARIABLE; gfc_conv_expr (&comp_se, e2); + e2->expr_type = old_type; gfc_free_expr (e2); return build_fold_addr_expr_loc (input_location, comp_se.expr); }