diff options
| author | burnus <burnus@138bc75d-0d04-0410-961f-82ee72b054a4> | 2008-09-24 07:01:18 +0000 |
|---|---|---|
| committer | burnus <burnus@138bc75d-0d04-0410-961f-82ee72b054a4> | 2008-09-24 07:01:18 +0000 |
| commit | 62754d540b5a88cea4a4eafe2fa90cbb3826bca8 (patch) | |
| tree | c0506931aec66b48c8d93f6b98854e14ef6028c2 | |
| parent | ad55073aebc429487335e8e8b1c07013cb4d2736 (diff) | |
| download | ppe42-gcc-62754d540b5a88cea4a4eafe2fa90cbb3826bca8.tar.gz ppe42-gcc-62754d540b5a88cea4a4eafe2fa90cbb3826bca8.zip | |
2008-09-24 Tobias Burnus <burnus@net-b.de>
PR fortran/37626
* trans-array.c (gfc_trans_deferred_array): Don't auto-deallocate
result variables.
2008-09-24 Tobias Burnus <burnus@net-b.de>
PR fortran/37626
* gfortran.dg/allocatable_function_4.f90: New test.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@140624 138bc75d-0d04-0410-961f-82ee72b054a4
| -rw-r--r-- | gcc/fortran/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/fortran/trans-array.c | 2 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gfortran.dg/allocatable_function_4.f90 | 56 |
4 files changed, 68 insertions, 1 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 1210d393efc..3d99ae35e70 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,9 @@ +2008-09-24 Tobias Burnus <burnus@net-b.de> + + PR fortran/37626 + * trans-array.c (gfc_trans_deferred_array): Don't auto-deallocate + result variables. + 2008-09-23 Daniel Kraft <d@domob.eu> PR fortran/37588 diff --git a/gcc/fortran/trans-array.c b/gcc/fortran/trans-array.c index 42b9967764f..c5aff65eeaa 100644 --- a/gcc/fortran/trans-array.c +++ b/gcc/fortran/trans-array.c @@ -5754,7 +5754,7 @@ gfc_trans_deferred_array (gfc_symbol * sym, tree body) gfc_add_expr_to_block (&fnblock, tmp); } - if (sym->attr.allocatable && !sym->attr.save) + if (sym->attr.allocatable && !sym->attr.save && !sym->attr.result) { tmp = gfc_trans_dealloc_allocated (sym->backend_decl); gfc_add_expr_to_block (&fnblock, tmp); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 411cf0c56a1..aa736dc4aae 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2008-09-24 Tobias Burnus <burnus@net-b.de> + + PR fortran/37626 + * gfortran.dg/allocatable_function_4.f90: New test. + 2008-09-23 Steve Ellcey <sje@cup.hp.com> * lib/target-supports.exp (check_effective_target_pow10): New. diff --git a/gcc/testsuite/gfortran.dg/allocatable_function_4.f90 b/gcc/testsuite/gfortran.dg/allocatable_function_4.f90 new file mode 100644 index 00000000000..9aff3a85a2d --- /dev/null +++ b/gcc/testsuite/gfortran.dg/allocatable_function_4.f90 @@ -0,0 +1,56 @@ +! { dg-do compile } +! { dg-options "-fdump-tree-original" } +! +! PR fortran/37626 +! Contributed by Rich Townsend +! +! The problem was an ICE when trying to deallocate the +! result variable "x_unique". +! +function unique_A (x, sorted) result (x_unique) + implicit none + character(*), dimension(:), intent(in) :: x + logical, intent(in), optional :: sorted + character(LEN(x)), dimension(:), allocatable :: x_unique + + logical :: sorted_ + character(LEN(x)), dimension(SIZE(x)) :: x_sorted + integer :: n_x + logical, dimension(SIZE(x)) :: mask + + integer, external :: b3ss_index + +! Set up sorted_ + + if(PRESENT(sorted)) then + sorted_ = sorted + else + sorted_ = .FALSE. + endif + +! If necessary, sort x + + if(sorted_) then + x_sorted = x + else + x_sorted = x(b3ss_index(x)) + endif + +! Set up the unique array + + n_x = SIZE(x) + + mask = (/.TRUE.,x_sorted(2:n_x) /= x_sorted(1:n_x-1)/) + + allocate(x_unique(COUNT(mask))) + + x_unique = PACK(x_sorted, MASK=mask) + +! Finish + + return +end function unique_A + +! { dg-final { scan-tree-dump-times "__builtin_free" 5 "original" } } +! { dg-final { cleanup-tree-dump "original" } } + |
