public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length
@ 2020-09-01 19:53 ajmay81 at googlemail dot com
  2020-09-02  7:06 ` [Bug libfortran/96886] [10 Regression] " marxin at gcc dot gnu.org
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: ajmay81 at googlemail dot com @ 2020-09-01 19:53 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96886

            Bug ID: 96886
           Summary: valgrind error with optional character argument of
                    unknown length
           Product: gcc
           Version: 10.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libfortran
          Assignee: unassigned at gcc dot gnu.org
          Reporter: ajmay81 at googlemail dot com
  Target Milestone: ---

Looks like a regression between 10.1.0 and 10.2.0. test.f90:

module test_module
  implicit none
contains
  subroutine testsub0(propnam)
    character(*), intent(in), optional :: propnam(*)
    if(.not.present(propnam)) then
       stop 'property names not given'
    else
       write(*,*) propnam(1)
    endif
  end subroutine testsub0
end module test_module

program bug
  use test_module
  implicit none
  character(8) :: prop(1)
  prop(1)='POT'
  call testsub0(propnam=prop)
end program bug

bug> module load gcc/10.2.0
bug> gfortran test.f90 && valgrind ./a.out
==31524== Memcheck, a memory error detector
==31524== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==31524== Using Valgrind-3.15.0 and LibVEX; rerun with -h for copyright info
==31524== Command: ./a.out
==31524==
==31524== Conditional jump or move depends on uninitialised value(s)
==31524==    at 0x400741: __test_module_MOD_testsub0 (in /home/andy/bug/a.out)
==31524==    by 0x400825: MAIN__ (in /home/andy/bug/a.out)
==31524==    by 0x40085C: main (in /home/andy/bug/a.out)
==31524== 
==31524== Conditional jump or move depends on uninitialised value(s)
==31524==    at 0x400762: __test_module_MOD_testsub0 (in /home/andy/bug/a.out)
==31524==    by 0x400825: MAIN__ (in /home/andy/bug/a.out)
==31524==    by 0x40085C: main (in /home/andy/bug/a.out)
==31524== 
STOP property names not given
==31524== 
==31524== HEAP SUMMARY:
==31524==     in use at exit: 0 bytes in 0 blocks
==31524==   total heap usage: 21 allocs, 21 frees, 13,584 bytes allocated
==31524==
==31524== All heap blocks were freed -- no leaks are possible
==31524== 
==31524== Use --track-origins=yes to see where uninitialised values come from
==31524== For lists of detected and suppressed errors, rerun with: -s
==31524== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 0 from 0)

bug> module purge
bug> module load gcc/10.1.0 
bug> gfortran test.f90 && valgrind ./a.out
==31516== Memcheck, a memory error detector
==31516== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==31516== Using Valgrind-3.15.0 and LibVEX; rerun with -h for copyright info
==31516== Command: ./a.out
==31516== 
 POT     
==31516== 
==31516== HEAP SUMMARY:
==31516==     in use at exit: 0 bytes in 0 blocks
==31516==   total heap usage: 21 allocs, 21 frees, 13,584 bytes allocated
==31516== 
==31516== All heap blocks were freed -- no leaks are possible
==31516== 
==31516== For lists of detected and suppressed errors, rerun with: -s
==31516== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug libfortran/96886] [10 Regression] valgrind error with optional character argument of unknown length
  2020-09-01 19:53 [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length ajmay81 at googlemail dot com
@ 2020-09-02  7:06 ` marxin at gcc dot gnu.org
  2020-09-02  7:42 ` [Bug fortran/96886] " rguenth at gcc dot gnu.org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-09-02  7:06 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96886

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW
                 CC|                            |burnus at gcc dot gnu.org,
                   |                            |jakub at gcc dot gnu.org,
                   |                            |marxin at gcc dot gnu.org
   Last reconfirmed|                            |2020-09-02
            Summary|valgrind error with         |[10 Regression] valgrind
                   |optional character argument |error with optional
                   |of unknown length           |character argument of
                   |                            |unknown length

--- Comment #1 from Martin Liška <marxin at gcc dot gnu.org> ---
Confirmed, started with r10-8137-g8aeffc4c9f47dd09ccc6a82d9fae35931414eb4d.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/96886] [10 Regression] valgrind error with optional character argument of unknown length
  2020-09-01 19:53 [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length ajmay81 at googlemail dot com
  2020-09-02  7:06 ` [Bug libfortran/96886] [10 Regression] " marxin at gcc dot gnu.org
@ 2020-09-02  7:42 ` rguenth at gcc dot gnu.org
  2020-09-02  7:51 ` marxin at gcc dot gnu.org
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: rguenth at gcc dot gnu.org @ 2020-09-02  7:42 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96886

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |10.3
           Priority|P3                          |P4
          Component|libfortran                  |fortran

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/96886] [10 Regression] valgrind error with optional character argument of unknown length
  2020-09-01 19:53 [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length ajmay81 at googlemail dot com
  2020-09-02  7:06 ` [Bug libfortran/96886] [10 Regression] " marxin at gcc dot gnu.org
  2020-09-02  7:42 ` [Bug fortran/96886] " rguenth at gcc dot gnu.org
@ 2020-09-02  7:51 ` marxin at gcc dot gnu.org
  2020-09-02  8:04 ` burnus at gcc dot gnu.org
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-09-02  7:51 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96886

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |10.2.0
      Known to work|                            |10.1.0, 11.0
            Summary|[10/11 Regression] valgrind |[10 Regression] valgrind
                   |error with optional         |error with optional
                   |character argument of       |character argument of
                   |unknown length              |unknown length

--- Comment #2 from Martin Liška <marxin at gcc dot gnu.org> ---
Apparently, the current master is fine.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/96886] [10 Regression] valgrind error with optional character argument of unknown length
  2020-09-01 19:53 [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length ajmay81 at googlemail dot com
                   ` (2 preceding siblings ...)
  2020-09-02  7:51 ` marxin at gcc dot gnu.org
@ 2020-09-02  8:04 ` burnus at gcc dot gnu.org
  2020-10-31 12:17 ` pault at gcc dot gnu.org
  2020-11-02  9:03 ` marxin at gcc dot gnu.org
  5 siblings, 0 replies; 7+ messages in thread
From: burnus at gcc dot gnu.org @ 2020-09-02  8:04 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96886

--- Comment #3 from Tobias Burnus <burnus at gcc dot gnu.org> ---
(In reply to Andy May from comment #0)
> Looks like a regression between 10.1.0 and 10.2.0. test.f90:

I think that's the same issue as reported in PR 94672 comment 12 last week –
which has been fixed very recently: last Friday in r11-2923 and Monday, three
days ago, in r10-8691.

Thus, I believe it is fine now. – Thanks for the report and sorry for the
breakage.

(In reply to Martin Liška from comment #2)
> Apparently, the current master is fine.

Unsurprisingly, given that I has been fixed just a few days ago. You need to
update your bisecter :-) And thanks for doing all this bisecting and bug
triages!

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/96886] [10 Regression] valgrind error with optional character argument of unknown length
  2020-09-01 19:53 [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length ajmay81 at googlemail dot com
                   ` (3 preceding siblings ...)
  2020-09-02  8:04 ` burnus at gcc dot gnu.org
@ 2020-10-31 12:17 ` pault at gcc dot gnu.org
  2020-11-02  9:03 ` marxin at gcc dot gnu.org
  5 siblings, 0 replies; 7+ messages in thread
From: pault at gcc dot gnu.org @ 2020-10-31 12:17 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96886

Paul Thomas <pault at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pault at gcc dot gnu.org

--- Comment #4 from Paul Thomas <pault at gcc dot gnu.org> ---
I presume that this one can be closed then?

Paul

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/96886] [10 Regression] valgrind error with optional character argument of unknown length
  2020-09-01 19:53 [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length ajmay81 at googlemail dot com
                   ` (4 preceding siblings ...)
  2020-10-31 12:17 ` pault at gcc dot gnu.org
@ 2020-11-02  9:03 ` marxin at gcc dot gnu.org
  5 siblings, 0 replies; 7+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-11-02  9:03 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96886

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Martin Liška <marxin at gcc dot gnu.org> ---
Closing then.

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-11-02  9:03 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-01 19:53 [Bug libfortran/96886] New: valgrind error with optional character argument of unknown length ajmay81 at googlemail dot com
2020-09-02  7:06 ` [Bug libfortran/96886] [10 Regression] " marxin at gcc dot gnu.org
2020-09-02  7:42 ` [Bug fortran/96886] " rguenth at gcc dot gnu.org
2020-09-02  7:51 ` marxin at gcc dot gnu.org
2020-09-02  8:04 ` burnus at gcc dot gnu.org
2020-10-31 12:17 ` pault at gcc dot gnu.org
2020-11-02  9:03 ` marxin at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).