From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.smtpout.orange.fr (smtp02.smtpout.orange.fr [80.12.242.124]) by sourceware.org (Postfix) with ESMTPS id 6B7053858428 for ; Tue, 14 Dec 2021 11:50:28 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 6B7053858428 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] ([86.253.179.215]) by smtp.orange.fr with ESMTPA id x6K0mAr7GqYovx6KAm9zVS; Tue, 14 Dec 2021 12:50:27 +0100 X-ME-Helo: [192.168.1.17] X-ME-Auth: MDU4MTIxYWM4YWI0ZGE4ZTUwZWZmNTExZmI2ZWZlMThkM2ZhYiE5OWRkOGM= X-ME-Date: Tue, 14 Dec 2021 12:50:27 +0100 X-ME-IP: 86.253.179.215 Message-ID: Date: Tue, 14 Dec 2021 12:50:16 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.4.0 Subject: Re: [PATCH, v2] PR libfortran/103634 - Runtime crash with PACK on zero-sized arrays Content-Language: fr To: Harald Anlauf , fortran@gcc.gnu.org Cc: gcc-patches@gcc.gnu.org References: <6c67790c-bda2-299b-a253-dc96bb11fea0@orange.fr> <44f97688-e1be-e23e-9480-7fc78b3b6d78@gmx.de> From: Mikael Morin In-Reply-To: <44f97688-e1be-e23e-9480-7fc78b3b6d78@gmx.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 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: Tue, 14 Dec 2021 11:50:30 -0000 Le 13/12/2021 à 21:27, Harald Anlauf via Fortran a écrit : > Works better with patch attached... > > Am 13.12.21 um 21:25 schrieb Harald Anlauf via Gcc-patches: >> >> The code is so similar (for good reason) that it makes sense to keep >> it synchronous.  I added code for 'zero_sized' array with the minor >> difference that I made it boolean instead of integer. >> >> I also extended the testcase so that it exercises PACK/pack_internal >> a little, for argument 'vector' present as well as not.  (There are >> existing tests for intrinsic types, but not for the issue at hand). >> >> Regtested again, and checked the testcase (against other compilers >> and also with valgrind). >> >> OK now? >> Yes, thanks.