[Bug fortran/100040] New: Wrong code with intent out assumed-rank allocatable

[Bug fortran/100040] New: Wrong code with intent out assumed-rank allocatable

[jrfsousa at gmail dot com]

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

            Bug ID: 100040
           Summary: Wrong code with intent out assumed-rank allocatable
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: fortran
          Assignee: unassigned at gcc dot gnu.org
          Reporter: jrfsousa at gmail dot com
  Target Milestone: ---

Created attachment 50562
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50562&action=edit
Fortran code showing problem

Hi All!

With intent out assumed-rank allocatable argument the generated code is missing
is missing the descriptor copy out.

Seen in:

GNU Fortran (GCC) 11.0.1 20210411 (experimental)
GNU Fortran (GCC) 10.3.1 20210411

Thank you very much.

Best regards,
José Rui

[Bug fortran/100040] Wrong code with intent out assumed-rank allocatable

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2021-04-14
     Ever confirmed|0                           |1

--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
Confirmed.

[Bug fortran/100040] Wrong code with intent out assumed-rank allocatable

[jrfsousa at gmail dot com]

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

--- Comment #2 from José Rui Faustino de Sousa <jrfsousa at gmail dot com> ---
Patch posted.

https://gcc.gnu.org/pipermail/fortran/2021-April/055924.html

[Bug fortran/100040] Wrong code with intent out assumed-rank allocatable

[cvs-commit at gcc dot gnu.org]

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

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Harald Anlauf <anlauf@gcc.gnu.org>:

https://gcc.gnu.org/g:5299155bb80e90df822e1eebc9f9a0c8e4505a46

commit r13-3014-g5299155bb80e90df822e1eebc9f9a0c8e4505a46
Author: José Rui Faustino de Sousa <jrfsousa@gmail.com>
Date:   Sun Sep 25 22:48:55 2022 +0200

    Fortran: Fix ICE and wrong code for assumed-rank arrays [PR100029,
PR100040]

    gcc/fortran/ChangeLog:

            PR fortran/100040
            PR fortran/100029
            * trans-expr.cc (gfc_conv_class_to_class): Add code to have
            assumed-rank arrays recognized as full arrays and fix the type
            of the array assignment.
            (gfc_conv_procedure_call): Change order of code blocks such that
            the free of ALLOCATABLE dummy arguments with INTENT(OUT) occurs
            first.

    gcc/testsuite/ChangeLog:

            PR fortran/100029
            * gfortran.dg/PR100029.f90: New test.

            PR fortran/100040
            * gfortran.dg/PR100040.f90: New test.

[Bug fortran/100040] Wrong code with intent out assumed-rank allocatable

[cvs-commit at gcc dot gnu.org]

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

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-12 branch has been updated by Harald Anlauf
<anlauf@gcc.gnu.org>:

https://gcc.gnu.org/g:07da38566232cc11901c3eeb9fc2860f02f2e31f

commit r12-8813-g07da38566232cc11901c3eeb9fc2860f02f2e31f
Author: José Rui Faustino de Sousa <jrfsousa@gmail.com>
Date:   Sun Sep 25 22:48:55 2022 +0200

    Fortran: Fix ICE and wrong code for assumed-rank arrays [PR100029,
PR100040]

    gcc/fortran/ChangeLog:

            PR fortran/100040
            PR fortran/100029
            * trans-expr.cc (gfc_conv_class_to_class): Add code to have
            assumed-rank arrays recognized as full arrays and fix the type
            of the array assignment.
            (gfc_conv_procedure_call): Change order of code blocks such that
            the free of ALLOCATABLE dummy arguments with INTENT(OUT) occurs
            first.

    gcc/testsuite/ChangeLog:

            PR fortran/100029
            * gfortran.dg/PR100029.f90: New test.

            PR fortran/100040
            * gfortran.dg/PR100040.f90: New test.

    (cherry picked from commit 5299155bb80e90df822e1eebc9f9a0c8e4505a46)

[Bug fortran/100040] Wrong code with intent out assumed-rank allocatable

[cvs-commit at gcc dot gnu.org]

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

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Harald Anlauf
<anlauf@gcc.gnu.org>:

https://gcc.gnu.org/g:1167c14d598021581e984cbe46273ade4bc54126

commit r11-10295-g1167c14d598021581e984cbe46273ade4bc54126
Author: José Rui Faustino de Sousa <jrfsousa@gmail.com>
Date:   Sun Sep 25 22:48:55 2022 +0200

    Fortran: Fix ICE and wrong code for assumed-rank arrays [PR100029,
PR100040]

    gcc/fortran/ChangeLog:

            PR fortran/100040
            PR fortran/100029
            * trans-expr.c (gfc_conv_class_to_class): Add code to have
            assumed-rank arrays recognized as full arrays and fix the type
            of the array assignment.
            (gfc_conv_procedure_call): Change order of code blocks such that
            the free of ALLOCATABLE dummy arguments with INTENT(OUT) occurs
            first.

    gcc/testsuite/ChangeLog:

            PR fortran/100029
            * gfortran.dg/PR100029.f90: New test.

            PR fortran/100040
            * gfortran.dg/PR100040.f90: New test.

    (cherry picked from commit 5299155bb80e90df822e1eebc9f9a0c8e4505a46)

[Bug fortran/100040] Wrong code with intent out assumed-rank allocatable

[cvs-commit at gcc dot gnu.org]

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

--- Comment #6 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Harald Anlauf
<anlauf@gcc.gnu.org>:

https://gcc.gnu.org/g:a880f9a44ecad385d4273f96f76c5c900d716941

commit r10-11024-ga880f9a44ecad385d4273f96f76c5c900d716941
Author: José Rui Faustino de Sousa <jrfsousa@gmail.com>
Date:   Sun Sep 25 22:48:55 2022 +0200

    Fortran: Fix ICE and wrong code for assumed-rank arrays [PR100029,
PR100040]

    gcc/fortran/ChangeLog:

            PR fortran/100040
            PR fortran/100029
            * trans-expr.c (gfc_conv_class_to_class): Add code to have
            assumed-rank arrays recognized as full arrays and fix the type
            of the array assignment.
            (gfc_conv_procedure_call): Change order of code blocks such that
            the free of ALLOCATABLE dummy arguments with INTENT(OUT) occurs
            first.

    gcc/testsuite/ChangeLog:

            PR fortran/100029
            * gfortran.dg/PR100029.f90: New test.

            PR fortran/100040
            * gfortran.dg/PR100040.f90: New test.

    (cherry picked from commit 5299155bb80e90df822e1eebc9f9a0c8e4505a46)

[Bug fortran/100040] Wrong code with intent out assumed-rank allocatable

[anlauf at gcc dot gnu.org]

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

anlauf at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |anlauf at gcc dot gnu.org
             Status|NEW                         |RESOLVED
   Target Milestone|---                         |10.5
         Resolution|---                         |FIXED

--- Comment #7 from anlauf at gcc dot gnu.org ---
Fixed on all open branches.  Closing.

Thanks for the patch!