diff options
| author | Tobias Burnus <burnus@net-b.de> | 2010-12-18 21:18:43 +0100 |
|---|---|---|
| committer | Tobias Burnus <burnus@gcc.gnu.org> | 2010-12-18 21:18:43 +0100 |
| commit | b5dca6ea71c1de8775d723d5af41618c3581ae68 (patch) | |
| tree | 371351ec73c66d702a7b86022c042d333f6bfcce /gcc/fortran/trans-expr.c | |
| parent | a93bb2bc65a048efadf16df300dbab2d4adfc8ae (diff) | |
| download | gcc-b5dca6ea71c1de8775d723d5af41618c3581ae68.zip gcc-b5dca6ea71c1de8775d723d5af41618c3581ae68.tar.gz gcc-b5dca6ea71c1de8775d723d5af41618c3581ae68.tar.bz2 | |
re PR fortran/46974 (ICE with TRANSFER using a C_PTR entity)
2010-12-18 Tobias Burnus <burnus@net-b.de>
PR fortran/46974
* target-memory.c (gfc_interpret_derived): Handle
* C_PTR/C_FUNPTR.
* trans-expr.c (gfc_trans_structure_assign): Ditto.
(gfc_conv_expr): Avoid crashes using non-C_NULL_(FUN)PTR const expr.
2010-12-18 Tobias Burnus <burnus@net-b.de>
PR fortran/46974
* gfortran.dg/c_ptr_tests_16.f90: New.
From-SVN: r168031
Diffstat (limited to 'gcc/fortran/trans-expr.c')
| -rw-r--r-- | gcc/fortran/trans-expr.c | 38 |
1 files changed, 22 insertions, 16 deletions
diff --git a/gcc/fortran/trans-expr.c b/gcc/fortran/trans-expr.c index cf2cdb6..3e994aa 100644 --- a/gcc/fortran/trans-expr.c +++ b/gcc/fortran/trans-expr.c @@ -4514,6 +4514,24 @@ gfc_trans_structure_assign (tree dest, gfc_expr * expr) gfc_start_block (&block); cm = expr->ts.u.derived->components; + + if (expr->ts.u.derived->from_intmod == INTMOD_ISO_C_BINDING + && (expr->ts.u.derived->intmod_sym_id == ISOCBINDING_PTR + || expr->ts.u.derived->intmod_sym_id == ISOCBINDING_FUNPTR)) + { + gfc_se se, lse; + + gcc_assert (cm->backend_decl == NULL); + gfc_init_se (&se, NULL); + gfc_init_se (&lse, NULL); + gfc_conv_expr (&se, gfc_constructor_first (expr->value.constructor)->expr); + lse.expr = dest; + gfc_add_modify (&block, lse.expr, + fold_convert (TREE_TYPE (lse.expr), se.expr)); + + return gfc_finish_block (&block); + } + for (c = gfc_constructor_first (expr->value.constructor); c; c = gfc_constructor_next (c), cm = cm->next) { @@ -4521,20 +4539,6 @@ gfc_trans_structure_assign (tree dest, gfc_expr * expr) if (!c->expr) continue; - /* Handle c_null_(fun)ptr. */ - if (c && c->expr && c->expr->ts.is_iso_c) - { - field = cm->backend_decl; - tmp = fold_build3_loc (input_location, COMPONENT_REF, - TREE_TYPE (field), - dest, field, NULL_TREE); - tmp = fold_build2_loc (input_location, MODIFY_EXPR, TREE_TYPE (tmp), - tmp, fold_convert (TREE_TYPE (tmp), - null_pointer_node)); - gfc_add_expr_to_block (&block, tmp); - continue; - } - field = cm->backend_decl; tmp = fold_build3_loc (input_location, COMPONENT_REF, TREE_TYPE (field), dest, field, NULL_TREE); @@ -4664,8 +4668,10 @@ gfc_conv_expr (gfc_se * se, gfc_expr * expr) if (expr->ts.type == BT_DERIVED && expr->ts.u.derived && expr->ts.u.derived->attr.is_iso_c) { - if (expr->symtree->n.sym->intmod_sym_id == ISOCBINDING_NULL_PTR - || expr->symtree->n.sym->intmod_sym_id == ISOCBINDING_NULL_FUNPTR) + if (expr->expr_type == EXPR_VARIABLE + && (expr->symtree->n.sym->intmod_sym_id == ISOCBINDING_NULL_PTR + || expr->symtree->n.sym->intmod_sym_id + == ISOCBINDING_NULL_FUNPTR)) { /* Set expr_type to EXPR_NULL, which will result in null_pointer_node being used below. */ |