This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch] PR fortran/92470 Fixes for CFI_address
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Tobias Burnus <tobias at codesourcery dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, fortran <fortran at gcc dot gnu dot org>
- Date: Thu, 14 Nov 2019 01:31:56 +0100
- Subject: Re: [Patch] PR fortran/92470 Fixes for CFI_address
- References: <7277b261-b0c9-018a-c4f5-c4c9ae0c8362@codesourcery.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Nov 12, 2019 at 03:42:23PM +0100, Tobias Burnus wrote:
> 2019-12-11 Tobias Burnus <tobias@codesourcery.com>
>
> libgfortran/
> PR fortran/92470
> * runtime/ISO_Fortran_binding.c (CFI_address): Handle non-zero
> lower_bound; update error message.
> (CFI_allocate): Fix comment typo.
> (CFI_establish): Fix identation, fix typos, don't check values of 'dv'
> argument.
>
> gcc/testsuite/
> PR fortran/92470
> * gfortran.dg/ISO_Fortran_binding_17.c: New.
> * gfortran.dg/ISO_Fortran_binding_17.f90: New.
> * gfortran.dg/ISO_Fortran_binding_1.c (elemental_mult_c, allocate_c,
> section_c, select_part_c): Update for CFI_{address} changes;
> add asserts.
This broke on hosts where the system compiler doesn't have
ISO_Fortran_binding.h header installed (e.g. GCC 8 and earlier).
Fixed thusly, tested on x86_64-linux, committed to trunk as obvious.
2019-11-14 Jakub Jelinek <jakub@redhat.com>
* gfortran.dg/ISO_Fortran_binding_17.c: Include
../../../libgfortran/ISO_Fortran_binding.h rather than
ISO_Fortran_binding.h.
--- gcc/testsuite/gfortran.dg/ISO_Fortran_binding_17.c.jj 2019-11-13 10:54:37.081172852 +0100
+++ gcc/testsuite/gfortran.dg/ISO_Fortran_binding_17.c 2019-11-14 01:19:36.704285484 +0100
@@ -2,7 +2,7 @@
#include <stdio.h>
#include <assert.h>
-#include "ISO_Fortran_binding.h"
+#include "../../../libgfortran/ISO_Fortran_binding.h"
void Csub(const CFI_cdesc_t *, size_t, CFI_index_t invalid);
Jakub