From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mo4-p00-ob.smtp.rzone.de (mo4-p00-ob.smtp.rzone.de [85.215.255.20]) by sourceware.org (Postfix) with ESMTPS id D85C53858C60; Sat, 25 Feb 2023 22:59:14 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org D85C53858C60 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=tkoenig.net Authentication-Results: sourceware.org; spf=none smtp.mailfrom=tkoenig.net ARC-Seal: i=1; a=rsa-sha256; t=1677365953; cv=none; d=strato.com; s=strato-dkim-0002; b=GrpyC8Dupzw3zBNlw2f8lgRVrHgNuwcEQGAssmZHebhafXDOCUUBdjoSZOCrddVhZA eY4k490u2NTc+jeM1Fs0slaSMPi2aVLdITDSXgXxcP6hhEWb7tonOk7YqU8wxaax8/6g VBVuw6Cq1re0dDSqJZ4knYZ/BXG42PvN5fKALllT0ZrdByoiHPmTb6t5exysJk+4oT0Z 3UUm8egFqRFFU9De3idd+EvwM5RMdpjUCCIUAih06bKpWBxUb5GM02o/irJrW6TVuY/a KPW5/qhkEDr04blR5qjfNeA6v8Tm4OZiV4e+ZSxqXjFXuulZDCqP3fINbQ7aSESpYEbv QFaw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; t=1677365953; s=strato-dkim-0002; d=strato.com; h=In-Reply-To:From:References:Cc:To:Subject:Date:Message-ID:Cc:Date: From:Subject:Sender; bh=ulsCJYDx1XPRY7ZtUtPKxd3N5kJuyr36gl5B3iNTugM=; b=d1tpDnS2RZePw7XmwfuOhy3sebVH8UNGDSHfK/3j9XK7yLFeJYODlqFlCbJXo3gN6v qF3P+mXLTmPaXBbcBT6EOtro+rHqTNhmWXYTLNrUfoBkfRjOIS2bw0juciBr+kpH9uOS bjkYIekwYt7SlZ35/vyBn2rCvI3y//Oybb2fwAGYLTbozBUG5dfXG84ZOZEKdKFUK8QU O61yMKYRn2qnPMU03B70ILOXWkxyfedUOD0XA/xwOkQ9WzOUoFcbdFnzevfQ3NOaF60W LRowojXH6LRDoRPd6KXrYeL82VYLGMF4/FxSp3ES3Mb4GwhZu3UkDhcHtyHk8LawFUV1 RoMw== ARC-Authentication-Results: i=1; strato.com; arc=none; dkim=none X-RZG-CLASS-ID: mo00 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1677365953; s=strato-dkim-0002; d=tkoenig.net; h=In-Reply-To:From:References:Cc:To:Subject:Date:Message-ID:Cc:Date: From:Subject:Sender; bh=ulsCJYDx1XPRY7ZtUtPKxd3N5kJuyr36gl5B3iNTugM=; b=N3ye/DPnfFCmyCq7U9uOUj+mva91xBL7yHEwjgd92hfvlHvJ5pkNN8sdxS6P2Hk1pj 9jVEfSo7tg6s93fkta8dA/Php/3LfhgvlzheU8n05gZDHz0PrVZw3SKBd0Nv5GfDYGDQ 2DqBFCqYyX4VjuMaS6lhNay04aVzqwj4kECwM2eU0VY+f9REDA+h3uTGtQvoGBKnfJZF 5uB+sHDRJksLRWl0QDAmL+JtAaPdnrr157CC2l7uMMm4rBRYtgvhzTdmvjJZqIbdBxJr Wz7fISFjuILVQTzSGj3D5WXsWRyLYfxKGERs6XuhcjfRtGxKq1XBnCsDkvsRxAsGBWwO S+og== X-RZG-AUTH: ":OGckYUunfvGNVUL0FlRnC4eRM+bOwx0tUtYTrJ/xeZX+ZVM2s+qedn5zhTzISPXyujm5EgMzaJpcwieOssus7/NWTUZxCP8=" Received: from [IPV6:2a0a:a540:2c35:0:7285:c2ff:fe6c:992d] by smtp.strato.de (RZmta 49.3.0 AUTH) with ESMTPSA id z8360ez1PMxDiRt (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Sat, 25 Feb 2023 23:59:13 +0100 (CET) Message-ID: <1af6ee92-9eaf-482b-2515-bb9b639e6434@tkoenig.net> Date: Sat, 25 Feb 2023 23:59:12 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1 Subject: Re: drop -fdump-fortran-global ? [was: Re: [PATCH,FORTRAN] Fix memory leak of gsymbol] Content-Language: en-US To: Bernhard Reutner-Fischer , Harald Anlauf Cc: fortran@gcc.gnu.org, Thomas Koenig References: <20181021140434.30743-1-rep.dot.nop@gmail.com> <20211027234320.16fc848b@nbbrfq> <20211029012302.052fc4c4@nbbrfq> <20211030185256.2e1c4434@nbbrfq> <20211030235101.3d3e9041@nbbrfq> <2ffbe5d9-88d2-e9a9-4f2b-1c3944b87d76@gmx.de> <20211031212544.043daf3a@nbbrfq> <20230225225518.3567af8d@nbbrfq> From: =?UTF-8?Q?Thomas_K=c3=b6nig?= In-Reply-To: <20230225225518.3567af8d@nbbrfq> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-3.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_NONE,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 25.02.23 22:55, Bernhard Reutner-Fischer wrote: > but > should we really keep the option, if we don't even test basics and if it > was more a specific debug dump, from the looks? It is always possible to add test cases if testing is required, but I don't think this is necessary. We also do not have test cases for -fdump-fortran-original, which is also quite handy and which can find a lot of bugs. And yes, it is rather useful when debugging issues with global identifiers, which are a bit special in Fortran. As I wrote in my mail when submitting the patch, > While debugging it, I also put in an option to dump the global > symbol table to standard output. I have included this in this > patch because I think this may not be the last bug in that > area 😄 That hasn't changed, IMHO. Best regards Thomas