From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.smtpout.orange.fr (smtp10.smtpout.orange.fr [80.12.242.132]) by sourceware.org (Postfix) with ESMTPS id D09353858C27 for ; Sun, 21 Nov 2021 11:46:02 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org D09353858C27 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=orange.fr Authentication-Results: sourceware.org; spf=none smtp.mailfrom=orange.fr Received: from [192.168.1.17] ([92.167.144.168]) by smtp.orange.fr with ESMTPA id olIHmPnsG3ptZolIHmyAME; Sun, 21 Nov 2021 12:46:01 +0100 X-ME-Helo: [192.168.1.17] X-ME-Auth: MDU4MTIxYWM4YWI0ZGE4ZTUwZWZmNTExZmI2ZWZlMThkM2ZhYiE5OWRkOGM= X-ME-Date: Sun, 21 Nov 2021 12:46:01 +0100 X-ME-IP: 92.167.144.168 Subject: Re: [PATCH] PR fortran/87851 - [9/10/11/12 Regression] Wrong return type for len_trim To: Harald Anlauf , fortran , gcc-patches References: From: Mikael Morin Message-ID: <02b4c4b8-8e3b-ff2e-e5ac-537c1c857380@orange.fr> Date: Sun, 21 Nov 2021 12:46:01 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: fr Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-5.2 required=5.0 tests=BAYES_00, FREEMAIL_FROM, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, NICE_REPLY_A, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_NONE, 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: fortran@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Fortran mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 21 Nov 2021 11:46:04 -0000 Le 19/11/2021 à 20:47, Harald Anlauf via Fortran a écrit : > Dear Fortranners, > > scalariziation of the elemental intrinsic LEN_TRIM was ICEing > when the optional KIND argument was present. > > The cleanest solution is to use the infrastructure added by > Mikael's fix for PR97896. In that case it is a 1-liner. :-) > > That fix is available on mainline and on 11-branch only, though. > My suggestion is to fix the current PR only for the same branches, > leaving the regression unfixed for older ones. > > Regtested on x86_64-pc-linux-gnu. OK for mainline and 11-branch? > Your change itself is fine. The PR was originally about a type mismatch between the gfortran library and the call generated by the front-end. As the code generated contains a cast, I think it’s fine as well. But please give Thomas (bug reporter) one more day to comment on this. Then I think you can proceed. Thanks.