From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from esa4.mentor.iphmx.com (esa4.mentor.iphmx.com [68.232.137.252]) by sourceware.org (Postfix) with ESMTPS id DE2093857C52; Fri, 22 Oct 2021 13:07:04 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org DE2093857C52 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mentor.com IronPort-SDR: kMxivSIdqda/AVjgX1EFIxQKLmyMMyi/bgIzZwTv+zo4Fh3P8fS/vaQQCZ7+PkO7crh6zOTkd8 hag4yzeI/Hy5koOBCxeb/sjW8HoQgk7Al0ol0GSTbHkDh5P2CVPtKC8VIqx6ljxQPGggVh8wS9 0wA8MFeXa3z7s+1tyfVWQYBLffwdaF2qoAHjPDrzy/ZvD7ASsi4gIBHDPWq0TKOXx+FaPzzvKP arItlYLi3iXXfd0ZMnPwXSvkOtwlMumtgC9xfFMooNalzA1Xt1xcbDlGRP+vGatkzZTfIW23ly iSGd+49ViM8qE2RmctFqvZW/ X-IronPort-AV: E=Sophos;i="5.87,172,1631606400"; d="scan'208";a="67526469" Received: from orw-gwy-02-in.mentorg.com ([192.94.38.167]) by esa4.mentor.iphmx.com with ESMTP; 22 Oct 2021 05:07:03 -0800 IronPort-SDR: GXFZUixgugtSPvz1vOyD9ag/4AD2QISun30qttnw58+WzrmX0Ng1M+SAz5yq2uEYWZbzBSFC1h CFDzggwv5q/h5xJxLj1oN71TZi0FCB3xnkNUvtm+cOSShCGy+72NedEqdoMhglwciWeuOdeogT mtbC6s46h+v2Wxq4XURnryHMKciIIlLTVrBtMGtDJGfNSRxcy1K+SUCRG8lbVYJkAw9/1EiO4N mGXryiLNII1xkhc3ZbKqYdtm3NkXU6UHCAG0seqCJ1EBcgvhcpENFBF2zPzdUKxdZE44fxfF5a Jh4= Message-ID: Date: Fri, 22 Oct 2021 15:06:45 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.1 Subject: =?UTF-8?Q?Re=3a_Jos=c3=a9=27s_pending_bind=28C=29_patches_/_status_?= =?UTF-8?Q?=28was=3a_Re=3a_=5bPatch=2c_fortran_V3=5d_PR_fortran/100683_-_Arr?= =?UTF-8?Q?ay_initialization_refuses_valid=29?= Content-Language: en-US From: Tobias Burnus To: =?UTF-8?Q?Jos=c3=a9_Rui_Faustino_de_Sousa?= , , CC: Sandra Loosemore , Paul Richard Thomas , Thomas Koenig , Harald Anlauf References: <01c32a3d-7a33-1866-667c-04689c0ae173@gmail.com> <327ebcb4-8502-ab31-ffda-e0be0740fcfc@codesourcery.com> In-Reply-To: <327ebcb4-8502-ab31-ffda-e0be0740fcfc@codesourcery.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: quoted-printable X-Originating-IP: [137.202.0.90] X-ClientProxiedBy: svr-ies-mbx-05.mgc.mentorg.com (139.181.222.5) To svr-ies-mbx-01.mgc.mentorg.com (139.181.222.1) X-Spam-Status: No, score=-6.5 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS, KAM_DMARC_STATUS, KAM_SHORT, NICE_REPLY_A, SPF_HELO_PASS, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 22 Oct 2021 13:07:07 -0000 Hi Jos=C3=A9, hi Fortraners, triage of all listed patches: > On 21.06.21 17:21, Jos=C3=A9 Rui Faustino de Sousa wrote: >> https://gcc.gnu.org/pipermail/fortran/2021-April/055924.html PR100040 - Wrong code with intent out assumed-rank allocatable PR100029 - ICE on subroutine call with allocatable polymorphic =E2=86=92 Both: Still occurs, ICE in gfc_deallocate_scalar_with_status TODO: Review patch. >> https://gcc.gnu.org/pipermail/fortran/2021-April/055933.html PR100097 - Unlimited polymorphic pointers and allocatables have incorrect r= ank PR100098 - Polymorphic pointers and allocatables have incorrect rank =E2=86=92 Both: PASS TODO: Check whether it makes sense to apply the testcase TODO: Close PRs =E2=86=92 See also patch below (2021-June/056169.html) >> https://gcc.gnu.org/pipermail/fortran/2021-June/056168.html PR96870 - Class name on error message =E2=86=92 Fixed with sufficient test coverage; thus, I closed the PR. Nothing to be done. >> https://gcc.gnu.org/pipermail/fortran/2021-June/056167.html PR96724 - Bogus warnings with the repeat intrinsic and the flag -Wconversio= n-extra|=C2=A0 repeat ('x', NCOPIES=3Di08) ! i08 is 20_1 shows: Warning: Co= nversion from INTEGER(1) to INTEGER(8) at (1) [-Wconversion-extra] TODO: Review patch. | >> https://gcc.gnu.org/pipermail/fortran/2021-June/056163.html Bug 93308 - bind(c) subroutine changes lower bound of array argument in cal= ler Bug 93963 - Select rank mishandling allocatable and pointer arguments with = bind(c) Bug 94327 - Bind(c) argument attributes are incorrectly set Bug 94331 - Bind(C) corrupts array descriptors Bug 97046 - Bad interaction between lbound/ubound, allocatable arrays and b= ind(C) subroutine with dimension(..) parameter =E2=86=92 All already closed as FIXED TODO: Nothing, unless we want to pick one of the testcases. >> https://gcc.gnu.org/pipermail/fortran/2021-June/056162.html PR94104 - Request for diagnostic improvement 10 | print *, sumf(a) | 1 Error: Actual argument for =E2=80=98a=E2=80=99 must be a pointer at (1) NOTE: as the dummy is intent(in), since F2008 alternatively a TARGET attr w= ould be also okay. TODO: Review patch - in principle, I am fine with the but I am not sure the 'valid target' in the error message is clear enough. Might require some mes= sage tweaking for clarity. >> https://gcc.gnu.org/pipermail/fortran/2021-June/056155.html Gerald's PR100948 - [12 Regression] ICE in gfc_conv_expr_val, at fortran/tr= ans-expr.c:9069 Still has an ICE. TODO: Review patch. >> https://gcc.gnu.org/pipermail/fortran/2021-June/056154.html Bug 100906 - Bind(c): failure handling character with len/=3D1 =E2=86=92 Testcase now passes. Bug 100907 - Bind(c): failure handling wide character =E2=86=92 I think now okay =E2=80=93 but the testcase assumes elem_len/size= of(char) =3D=3D #chars but for the C descriptor, elem_len / sizeof(char-type) =3D #chars Thus, sz is not 1 or 7 bytes but 4 or 28 bytes (or 1/7 characters) Bug 100911 - Bind(c): failure handling C_PTR =E2=86=92 Closed as FIXED. Bug 100914 - Bind(c): errors handling complex =E2=86=92 Closed as FIXED Bug 100915 - Bind(c): failure handling C_FUNPTR =E2=86=92 Closed as FIXED Bug 100916 - Bind(c): CFI_type_other unimplemented =E2=86=92 Bogus testcase (for 't(ype)' argument) otherwise it expects CFI_type_other instead of CFI_type_struct (TODO: Is that sensible?) TODO: Check whether a testcase is needed TODO: Close the three still open PRs >> https://gcc.gnu.org/pipermail/fortran/2021-June/056152.html Bug 101047 - Pointer explicit initialization fails Bug 101048 - Class pointer explicit initialization refuses valid ..., pointer, save :: ptr =3D> tgt fails to associate ptr with tgt (wrong-code + rejects valid) TODO: Review patch. >> https://gcc.gnu.org/pipermail/fortran/2021-June/056159.html PR92621 - Problems with memory handling with allocatable intent(out) arrays= with bind(c) I think mostly fixed by my big bind(C) patch, but there still one ICE with '-fcheck=3Dall -fsanitize=3Dundefined' TODO: Fix that bug (unlikely to be fixed by Jos=C3=A9's patch) TODO: Check whether testcase should be added and then close the PR >> https://gcc.gnu.org/pipermail/fortran/2021-April/055982.html PR100245 - ICE on automatic reallocation. Still ICEs TODO: Review patch. >> https://gcc.gnu.org/pipermail/fortran/2021-April/055949.html PR100136 - ICE, regression, using flag -fcheck=3Dpointer First testcase has an ICE with -fcheck=3Dpointer Second testcase has always an ICE + possibly missing func. TODO: Review patch =E2=80=93 and probably: follow-up patch for remaining is= sue >> https://gcc.gnu.org/pipermail/fortran/2021-April/055946.html PR100132 - Optimization breaks pointer association. 'fn spec' is wrong :-( TODO: Review patch! >> https://gcc.gnu.org/pipermail/fortran/2021-April/055934.html PR100103 - Automatic reallocation fails inside select rank Still segfaults at runtime for 'that =3D a' where the RHS is a parameter and the LHS an allocatable assumed-rank array (inside select rank). TODO: Review patch >> https://gcc.gnu.org/pipermail/fortran/2021-June/056169.html PR100097 - Unlimited polymorphic pointers and allocatables have incorrect r= ank PR100098 - Polymorphic pointers and allocatables have incorrect rank =E2=86=92 Both: PASS TODO: Check whether it makes sense to apply the testcase TODO: Close PRs =E2=86=92 See also patch above (2021-April/055933.html) >> https://gcc.gnu.org/pipermail/fortran/2021-April/055921.html PR100024 & PR100025 - ICE on missing polymorphic argument. ICE in argument_rank_mismatch TODO: Review patch. >> I am not 100% sure this is all of them but it should be most. If PRs link them, we find them eventually. Otherwise, we won't :-( Tobias ----------------- Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstra=C3=9Fe 201= , 80634 M=C3=BCnchen; Gesellschaft mit beschr=C3=A4nkter Haftung; Gesch=C3= =A4ftsf=C3=BChrer: Thomas Heurung, Frank Th=C3=BCrauf; Sitz der Gesellschaf= t: M=C3=BCnchen; Registergericht M=C3=BCnchen, HRB 106955