From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 97264 invoked by alias); 11 May 2019 07:54:23 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 97250 invoked by uid 89); 11 May 2019 07:54:22 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,KAM_NUMSUBJECT,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=no version=3.3.1 spammy=H*f:sk:z-Wjyov, H*f:sk:CAGkQGi, sincerely X-HELO: mail-lf1-f65.google.com Received: from mail-lf1-f65.google.com (HELO mail-lf1-f65.google.com) (209.85.167.65) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 11 May 2019 07:54:21 +0000 Received: by mail-lf1-f65.google.com with SMTP id v18so5711303lfi.1; Sat, 11 May 2019 00:54:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to; bh=BBlMVpNX7ed9qtDu+oAOsbGg80x6rdPf1g8h3RjAoxU=; b=ptyx2peLJYQGPJtyHGf4e6OMkOg9+Atwv4zxplIkkT89/Gw43qS8UFEa4pD4/FPh/D TiH2R+NXv361i3/cRW+QkktrC+tvsB9iBxk1LJ5WYmpNdWcu8NAee0rTE6i1+fKAtySr ObvXo8qgw9DGwxDBQ6X0rJHxOa6EYdZwEQ/IkAd0w3OK9UT3QndqfFDD1K7XaSZ3mc8E 7KMkEEoN/IRHaBBM/E2lSMQUXwwZoKgrxVpHEJznoaEvUGK4+ujS0YW5gV/tj3uCvo37 bis72XBFPCXgWR22VJVeOJigDvgNQd2KvPut5QILv8Ny2uJ6B4VVBggEy/4+2ET/mx8U 3PDQ== MIME-Version: 1.0 References: In-Reply-To: From: Paul Richard Thomas Date: Sat, 11 May 2019 07:54:00 -0000 Message-ID: Subject: Re: [Patch, fortran] ISO_Fortran_binding PRs 90093, 90352 & 90355 To: "fortran@gcc.gnu.org" , gcc-patches Content-Type: text/plain; charset="UTF-8" X-SW-Source: 2019-05/txt/msg00537.txt.bz2 Committed to 9-branch as revision 271089. Paul On Fri, 10 May 2019 at 09:00, Paul Richard Thomas wrote: > > Committed to trunk as revision 271057. > > Will do likewise with 9-branch asap. > > Cheers > > Paul > > On Wed, 8 May 2019 at 19:40, Paul Richard Thomas > wrote: > > > > Unless there are any objections to this patch, I plan to commit to > > trunk and 9-branch tomorrow night, with the change to the testcase > > pointed out by Dominique. > > > > I sincerely hope that will be the end of CFI PRs for a little while, > > at least. I have a load of pending patches and want to get on with > > fixing PDTs. > > > > Cheers > > > > Paul > > > > On Mon, 6 May 2019 at 19:59, Paul Richard Thomas > > wrote: > > > > > > It helps to attach the patch! > > > > > > On Mon, 6 May 2019 at 19:57, Paul Richard Thomas > > > wrote: > > > > > > > > Unfortunately, this patch was still in the making at the release of > > > > 9.1. It is more or less self explanatory with the ChangeLogs. > > > > > > > > It should be noted that gfc_conv_expr_present could not be used in the > > > > fix for PR90093 because the passed descriptor is a CFI type. Instead, > > > > the test is for a null pointer passed. > > > > > > > > The changes to trans-array.c(gfc_trans_create_temp_array) have an eye > > > > on the future, as well as PR90355. I am progressing towards the point > > > > where all descriptors have 'span' set correctly so that > > > > trans.c(get_array_span) can be eliminated and much of the code in the > > > > callers can be simplified. > > > > > > > > Bootstrapped and regtested on FC29/x86_64 - OK for trunk and 9-branch? > > > > > > > > Paul > > > > > > > > 2019-05-06 Paul Thomas > > > > > > > > PR fortran/90093 > > > > * trans-decl.c (convert_CFI_desc): Test that the dummy is > > > > present before doing any of the conversions. > > > > > > > > PR fortran/90352 > > > > * decl.c (gfc_verify_c_interop_param): Restore the error for > > > > charlen > 1 actual arguments passed to bind(C) procs. > > > > Clean up trailing white space. > > > > > > > > PR fortran/90355 > > > > * trans-array.c (gfc_trans_create_temp_array): Set the 'span' > > > > field to the element length for all types. > > > > (gfc_conv_expr_descriptor): The force_no_tmp flag is used to > > > > prevent temporary creation, especially for substrings. > > > > * trans-decl.c (gfc_trans_deferred_vars): Rather than assert > > > > that the backend decl for the string length is non-null, use it > > > > as a condition before calling gfc_trans_vla_type_sizes. > > > > * trans-expr.c (gfc_conv_gfc_desc_to_cfi_desc): 'force_no_tmp' > > > > is set before calling gfc_conv_expr_descriptor. > > > > * trans.c (get_array_span): Move the code for extracting 'span' > > > > from gfc_build_array_ref to this function. This is specific to > > > > descriptors that are component and indirect references. > > > > * trans.h : Add the force_no_tmp flag bitfield to gfc_se. > > > > > > > > 2019-05-06 Paul Thomas > > > > > > > > PR fortran/90093 > > > > * gfortran.dg/ISO_Fortran_binding_12.f90: New test. > > > > * gfortran.dg/ISO_Fortran_binding_12.c: Supplementary code. > > > > > > > > PR fortran/90352 > > > > * gfortran.dg/iso_c_binding_char_1.f90: New test. > > > > > > > > PR fortran/90355 > > > > * gfortran.dg/ISO_Fortran_binding_4.f90: Add 'substr' to test > > > > the direct passing of substrings as descriptors to bind(C). > > > > * gfortran.dg/assign_10.f90: Increase the tree_dump count of > > > > 'atmp' to account for the setting of the 'span' field. > > > > * gfortran.dg/transpose_optimization_2.f90: Ditto. > > > > > > > > -- > > "If you can't explain it simply, you don't understand it well enough" > > - Albert Einstein > > > > -- > "If you can't explain it simply, you don't understand it well enough" > - Albert Einstein -- "If you can't explain it simply, you don't understand it well enough" - Albert Einstein