diff options
author | Thomas Koenig <tkoenig@gcc.gnu.org> | 2016-10-30 17:48:27 +0000 |
---|---|---|
committer | Thomas Koenig <tkoenig@gcc.gnu.org> | 2016-10-30 17:48:27 +0000 |
commit | 396e56d2da7935f96f4ca133c359668cc6c98eaf (patch) | |
tree | 9f87be3a5ce110418b89e042a1508d940257fa15 | |
parent | e9d33505a09b5503ff8816ae970c47ce30684447 (diff) | |
download | gcc-396e56d2da7935f96f4ca133c359668cc6c98eaf.zip gcc-396e56d2da7935f96f4ca133c359668cc6c98eaf.tar.gz gcc-396e56d2da7935f96f4ca133c359668cc6c98eaf.tar.bz2 |
re PR fortran/67219 (Incorrect conversion warning)
2016-10-30 Thomas Koenig <tkoenig@gcc.gnu.org>
PR fortran/67219
* arith.c (gfc_int2real): Change gfc_warning_now
to gfc_warning.
* primary.c (match_complex_constant): If there
is no comma, throw away any warning which might have
been issued by gfc_int2real.
2016-10-30 Thomas Koenig <tkoenig@gcc.gnu.org>
PR fortran/67219
* gfortran.dg/pr67219.f90: New test.
From-SVN: r241689
-rw-r--r-- | gcc/fortran/ChangeLog | 9 | ||||
-rw-r--r-- | gcc/fortran/arith.c | 10 | ||||
-rw-r--r-- | gcc/fortran/primary.c | 4 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/gfortran.dg/pr67219.f90 | 8 |
5 files changed, 31 insertions, 5 deletions
diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 684af2f..be4f1b8 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,12 @@ +2016-10-30 Thomas Koenig <tkoenig@gcc.gnu.org> + + PR fortran/67219 + * arith.c (gfc_int2real): Change gfc_warning_now + to gfc_warning. + * primary.c (match_complex_constant): If there + is no comma, throw away any warning which might have + been issued by gfc_int2real. + 2016-10-28 Steven G. Kargl <kargl@gcc.gnu.org> PR fortran/71891 diff --git a/gcc/fortran/arith.c b/gcc/fortran/arith.c index 8af7540..d566faa 100644 --- a/gcc/fortran/arith.c +++ b/gcc/fortran/arith.c @@ -2072,11 +2072,11 @@ gfc_int2real (gfc_expr *src, int kind) if (warn_conversion && wprecision_int_real (src->value.integer, result->value.real)) - gfc_warning_now (OPT_Wconversion, "Change of value in conversion " - "from %qs to %qs at %L", - gfc_typename (&src->ts), - gfc_typename (&result->ts), - &src->where); + gfc_warning (OPT_Wconversion, "Change of value in conversion " + "from %qs to %qs at %L", + gfc_typename (&src->ts), + gfc_typename (&result->ts), + &src->where); return result; } diff --git a/gcc/fortran/primary.c b/gcc/fortran/primary.c index bcbaeaa..2101644 100644 --- a/gcc/fortran/primary.c +++ b/gcc/fortran/primary.c @@ -1353,6 +1353,10 @@ match_complex_constant (gfc_expr **result) if (gfc_match_char (',') == MATCH_NO) { + /* It is possible that gfc_int2real issued a warning when + converting an integer to real. Throw this away here. */ + + gfc_clear_warning (); gfc_pop_error (&old_error); m = MATCH_NO; goto cleanup; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 7ebc616..d65ebb4 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2016-10-30 Thomas Koenig <tkoenig@gcc.gnu.org> + + PR fortran/67219 + * gfortran.dg/pr67219.f90: New test. + 2016-10-29 Jakub Jelinek <jakub@redhat.com> PR rtl-optimization/77919 diff --git a/gcc/testsuite/gfortran.dg/pr67219.f90 b/gcc/testsuite/gfortran.dg/pr67219.f90 new file mode 100644 index 0000000..26d5f3f --- /dev/null +++ b/gcc/testsuite/gfortran.dg/pr67219.f90 @@ -0,0 +1,8 @@ +! { dg-do compile } +! PR 67149 - this used to throw a spurious error. +function foo(bar) + integer(8) :: foo + integer(4), intent(in) :: bar + integer(4), parameter :: huge_4 = huge(0_4) + foo = (huge_4 - int(bar,kind=8)) +end function |