diff options
| author | Louis Krupp <louis.krupp@zoho.com> | 2016-10-07 02:02:13 +0000 |
|---|---|---|
| committer | Louis Krupp <lkrupp@gcc.gnu.org> | 2016-10-07 02:02:13 +0000 |
| commit | ac193ee73ca85cae3e7c761f1bda491620b4ccab (patch) | |
| tree | 119ae41e8db5840e6f6ee3f69e192e2ea3f6b7d0 /gcc | |
| parent | fe37b0f08625cc57b5a92578dfa518100d3171e1 (diff) | |
| download | gcc-ac193ee73ca85cae3e7c761f1bda491620b4ccab.zip gcc-ac193ee73ca85cae3e7c761f1bda491620b4ccab.tar.gz gcc-ac193ee73ca85cae3e7c761f1bda491620b4ccab.tar.bz2 | |
2016_10-06 Louis Krupp <louis.krupp@zoho.com>
PR fortran/57910
* gfortran.dg/pr57910.f90: New test.
2016-10-05 Louis Krupp <louis.krupp@zoho.com>
PR fortran/57910
* trans-expr.c (gfc_add_interface_mapping): Don't try to
dereference call-by-value scalar argument
From-SVN: r240850
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/fortran/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/fortran/trans-expr.c | 4 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gfortran.dg/pr57910.f90 | 26 |
4 files changed, 41 insertions, 0 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 2bab7b3..874779b 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,9 @@ +2016-10-05 Louis Krupp <louis.krupp@zoho.com> + + PR fortran/57910 + * trans-expr.c (gfc_add_interface_mapping): Don't try to + dereference call-by-value scalar argument + 2016-10-05 Steven G. Kargl <kargls@gcc.gnu.org> PR fortran/58991 diff --git a/gcc/fortran/trans-expr.c b/gcc/fortran/trans-expr.c index a821788..1de2818 100644 --- a/gcc/fortran/trans-expr.c +++ b/gcc/fortran/trans-expr.c @@ -4009,6 +4009,10 @@ gfc_add_interface_mapping (gfc_interface_mapping * mapping, if (sym->attr.flavor == FL_PROCEDURE) value = se->expr; + /* If the argument is a pass-by-value scalar, use the value as is. */ + else if (!sym->attr.dimension && sym->attr.value) + value = se->expr; + /* If the argument is either a string or a pointer to a string, convert it to a boundless character type. */ else if (!sym->attr.dimension && sym->ts.type == BT_CHARACTER) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 9aee1b2..18c1a17 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2016_10-06 Louis Krupp <louis.krupp@zoho.com> + + PR fortran/57910 + * gfortran.dg/pr57910.f90: New test. + 2016-10-06 Jakub Jelinek <jakub@redhat.com> * g++.dg/cpp1z/has-unique-obj-representations1.C: New test. diff --git a/gcc/testsuite/gfortran.dg/pr57910.f90 b/gcc/testsuite/gfortran.dg/pr57910.f90 new file mode 100644 index 0000000..c3666a6 --- /dev/null +++ b/gcc/testsuite/gfortran.dg/pr57910.f90 @@ -0,0 +1,26 @@ +! { dg-do run } +program strtest + + implicit none + + character(len=:), allocatable:: my_str + + integer, parameter :: slen_init = 7 + integer :: slen = slen_init + + my_str = fstr(slen) + if (slen /= slen_init .or. len(my_str) /= slen .or. my_str /= ' ') then + call abort + endif + +contains + + function fstr(strlen) + integer, value :: strlen + character(len=strlen)::fstr + + strlen = 17 ! Make sure strlen was really passed by value + fstr = ' ' + end function fstr + +end program strtest |