git
/
gcc
mirror of git://gcc.gnu.org/git/gcc.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

re PR fortran/59345 (_gfortran_internal_pack on compiler generated temps) 

2019-01-11  Thomas Koenig  <tkoenig@gcc.gnu.org>

	PR fortran/59345
	* trans-array.c (gfc_conv_parameter_array):  Temporary
	arrays generated for expressions do not need to be repacked.

2019-01-11  Thomas Koenig  <tkoenig@gcc.gnu.org>

	PR fortran/59345
	* gfortran.dg/internal_pack_16.f90: New test.

From-SVN: r267829
This commit is contained in:
Thomas Koenig 2019-01-11 06:32:10 +00:00
parent 0604d5672f
commit 7dc3df082b
4 changed files with 26 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gcc/fortran/ChangeLog
View File

@ -1,3 +1,9 @@
2019-01-11 Thomas Koenig <tkoenig@gcc.gnu.org>
PR fortran/59345
* trans-array.c (gfc_conv_parameter_array): Temporary
arrays generated for expressions do not need to be repacked.
2019-01-10 Steven G. Kargl <kargl@gcc.gnu.org> 2019-01-10 Steven G. Kargl <kargl@gcc.gnu.org>
PR fortran/86322 PR fortran/86322

6
gcc/fortran/trans-array.c
View File

@ -7866,6 +7866,12 @@ gfc_conv_array_parameter (gfc_se * se, gfc_expr * expr, bool g77,
no_pack = contiguous && no_pack; no_pack = contiguous && no_pack;
/* If we have an expression, an array temporary will be
generated which does not need to be packed / unpacked
if passed to an explicit-shape dummy array. */
no_pack = no_pack || (g77 && expr->expr_type == EXPR_OP);
/* Array constructors are always contiguous and do not need packing. */ /* Array constructors are always contiguous and do not need packing. */
array_constructor = g77 && !this_array_result && expr->expr_type == EXPR_ARRAY; array_constructor = g77 && !this_array_result && expr->expr_type == EXPR_ARRAY;

5
gcc/testsuite/ChangeLog
View File

@ -1,3 +1,8 @@
2019-01-11 Thomas Koenig <tkoenig@gcc.gnu.org>
PR fortran/59345
* gfortran.dg/internal_pack_16.f90: New test.
2019-01-10 Jakub Jelinek <jakub@redhat.com> 2019-01-10 Jakub Jelinek <jakub@redhat.com>
PR target/88785 PR target/88785

9
gcc/testsuite/gfortran.dg/internal_pack_16.f90 Normal file
View File

@ -0,0 +1,9 @@
! { dg-do compile }
! { dg-additional-options "-fdump-tree-original" }
! PR 59345 - pack/unpack was not needed here.
SUBROUTINE S1(A)
REAL :: A(3)
CALL S2(-A)
END SUBROUTINE
! { dg-final { scan-tree-dump-not "_gfortran_internal_pack" "original" } }
! { dg-final { scan-tree-dump-not "_gfortran_internal_unpack" "original" } }