diff options
| author | Francois-Xavier Coudert <fxcoudert@gcc.gnu.org> | 2008-05-14 21:44:36 +0000 |
|---|---|---|
| committer | François-Xavier Coudert <fxcoudert@gcc.gnu.org> | 2008-05-14 21:44:36 +0000 |
| commit | 1c3339af10c27fafd9573c0a77c79144e23ebc51 (patch) | |
| tree | 1d799f9b345a459f5af51f936f09660ec4966c47 | |
| parent | 6401bf9cad029c264ff65db946c8e31ce998db13 (diff) | |
| download | gcc-1c3339af10c27fafd9573c0a77c79144e23ebc51.zip gcc-1c3339af10c27fafd9573c0a77c79144e23ebc51.tar.gz gcc-1c3339af10c27fafd9573c0a77c79144e23ebc51.tar.bz2 | |
re PR fortran/36059 (-frepack-arrays: symbols w/ TARGET should not be repacked)
PR fortran/36059
* trans-decl.c (gfc_build_dummy_array_decl): Don't repack
arrays that have the TARGET attribute.
* gfortran.dg/repack_arrays_1.f90: New test.
From-SVN: r135310
| -rw-r--r-- | gcc/fortran/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/fortran/trans-decl.c | 5 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gfortran.dg/repack_arrays_1.f90 | 24 |
4 files changed, 39 insertions, 1 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index c38717c..c39b86e 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,5 +1,11 @@ 2008-05-14 Francois-Xavier Coudert <fxcoudert@gcc.gnu.org> + PR fortran/36059 + * trans-decl.c (gfc_build_dummy_array_decl): Don't repack + arrays that have the TARGET attribute. + +2008-05-14 Francois-Xavier Coudert <fxcoudert@gcc.gnu.org> + PR fortran/36186 * simplify.c (only_convert_cmplx_boz): New function. (gfc_simplify_cmplx, gfc_simplify_complex, gfc_simplify_dcmplx): diff --git a/gcc/fortran/trans-decl.c b/gcc/fortran/trans-decl.c index 6e0b542..aa3712c 100644 --- a/gcc/fortran/trans-decl.c +++ b/gcc/fortran/trans-decl.c @@ -733,7 +733,10 @@ gfc_build_dummy_array_decl (gfc_symbol * sym, tree dummy) /* Create a descriptorless array pointer. */ as = sym->as; packed = PACKED_NO; - if (!gfc_option.flag_repack_arrays) + + /* Even when -frepack-arrays is used, symbols with TARGET attribute + are not repacked. */ + if (!gfc_option.flag_repack_arrays || sym->attr.target) { if (as->type == AS_ASSUMED_SIZE) packed = PACKED_FULL; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 9abe367..02ef0b5 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,10 @@ 2008-05-14 Francois-Xavier Coudert <fxcoudert@gcc.gnu.org> + PR fortran/36059 + * gfortran.dg/repack_arrays_1.f90: New test. + +2008-05-14 Francois-Xavier Coudert <fxcoudert@gcc.gnu.org> + PR fortran/36186 * gfortran.dg/boz_11.f90: New test. * gfortran.dg/boz_12.f90: New test. diff --git a/gcc/testsuite/gfortran.dg/repack_arrays_1.f90 b/gcc/testsuite/gfortran.dg/repack_arrays_1.f90 new file mode 100644 index 0000000..adf20aa --- /dev/null +++ b/gcc/testsuite/gfortran.dg/repack_arrays_1.f90 @@ -0,0 +1,24 @@ +! { dg-do run } +! { dg-options "-frepack-arrays" } +! +! Check that arrays marked with TARGET attribute are not repacked. +! +program test2 + use iso_c_binding + implicit none + real, target :: x(7) + type(c_ptr) cp1, cp2 + + x = 42 + if (.not. c_associated(c_loc(x(3)),point(x(::2)))) call abort +contains + function point(x) + use iso_c_binding + real, intent(in), target :: x(:) + type(c_ptr) point + real, pointer :: p + + p => x(2) + point = c_loc(p) + end function point +end program test2 |