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 C898C3858D35; Tue, 8 Mar 2022 20:19:52 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org C898C3858D35 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mentor.com X-IronPort-AV: E=Sophos;i="5.90,165,1643702400"; d="diff'?scan'208";a="72875773" Received: from orw-gwy-02-in.mentorg.com ([192.94.38.167]) by esa4.mentor.iphmx.com with ESMTP; 08 Mar 2022 12:19:52 -0800 IronPort-SDR: e1J3TV/kkSzA/DJPPhzs53PPKVFHp2ur9HDof0/VHHl/0gm74fsOBUHDMI2m+1zAS1xxlUu+6O K47sc6/uI+7cHBLhjlAEXkaOlIUhAeiiI84d/XWDhCAQxcWiBdwrPZ8LjFy15+cLRnmmmDI3DB Q+w5YbEM2rP3ONn6ZVNXVCSrdc7nQj/dBC/YW1KWVA6vDupls+X3ChiMV1urSBlz7q23v7+trX fJgDygakYg2ZhhG93KT4b5pWuGsQTd6LeSA20OW0mN4+xOMNjJmlNFum9R6OW8OXHmSCRd8Lpn TAw= Content-Type: multipart/mixed; boundary="------------TqsOHTwKiwhioUB8K1NFHB40" Message-ID: <7f7c728f-66d6-2eb6-b6ab-f271949926bf@codesourcery.com> Date: Tue, 8 Mar 2022 21:19:45 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.2 Subject: Re: [Patch] Fortran: Fix gfc_maybe_dereference_var [PR104430] Content-Language: en-US To: Harald Anlauf , gcc-patches , fortran References: <2cd2f3b9-3b67-dc5f-de37-94c9c5002e53@codesourcery.com> <7db8ae11-fd88-3ac6-e9fe-c8257e23bf6e@gmx.de> From: Tobias Burnus In-Reply-To: <7db8ae11-fd88-3ac6-e9fe-c8257e23bf6e@gmx.de> X-Originating-IP: [137.202.0.90] X-ClientProxiedBy: svr-ies-mbx-11.mgc.mentorg.com (139.181.222.11) To svr-ies-mbx-01.mgc.mentorg.com (139.181.222.1) X-Spam-Status: No, score=-11.5 required=5.0 tests=BAYES_00, GIT_PATCH_0, HEADER_FROM_DIFFERENT_DOMAINS, KAM_DMARC_STATUS, NICE_REPLY_A, SPF_HELO_PASS, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE 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, 08 Mar 2022 20:19:54 -0000 --------------TqsOHTwKiwhioUB8K1NFHB40 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: quoted-printable Hi Harald, On 07.03.22 20:58, Harald Anlauf wrote: > I think there are other PRs which profit from this fix. > Can you please have a look at PR99585, and in particular > the link in comment#0? ;-) Good pointer =E2=80=93 the testcase looks nearly identical and it is indeed= fixed. I included it in addition in the same testcase file. (See attached patch for the commit, .) Thanks, Tobias PS: Can I make you review my two pending patches? (NULL and SIZEOF) ;-) PPS: I lost a bit track working on other things =E2=80=93 are there patches pending review? PPPS: I think someone still has to deal with the approved and pending patches by Jos=C3=A9 ... ----------------- 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 --------------TqsOHTwKiwhioUB8K1NFHB40 Content-Type: text/x-patch; charset="UTF-8"; name="committed.diff" Content-Disposition: attachment; filename="committed.diff" Content-Transfer-Encoding: base64 Y29tbWl0IGMwMTM0YjczODM5OTJhYWI1YzFhOTE0NDBkYmRkOGZiYjc0NzE2OWMKQXV0aG9y OiBUb2JpYXMgQnVybnVzIDx0b2JpYXNAY29kZXNvdXJjZXJ5LmNvbT4KRGF0ZTogICBNb24g TWFyIDcgMjI6MTE6MzMgMjAyMiArMDEwMAoKICAgIEZvcnRyYW46IEZpeCBnZmNfbWF5YmVf ZGVyZWZlcmVuY2VfdmFyIFtQUjEwNDQzMF1bUFI5OTU4NV0KICAgIAogICAgICAgICAgICBQ UiBmb3J0cmFuLzk5NTg1CiAgICAgICAgICAgIFBSIGZvcnRyYW4vMTA0NDMwCiAgICAKICAg IGdjYy9mb3J0cmFuL0NoYW5nZUxvZzoKICAgIAogICAgICAgICAgICAqIHRyYW5zLWV4cHIu Y2MgKGNvbnZfcGFyZW50X2NvbXBvbmVudF9yZWZlcmVuY2VzKTogRml4IGNvbW1lbnQ7CiAg ICAgICAgICAgIHNpbXBsaWZ5IGNvbXBhcmlzb24uCiAgICAgICAgICAgIChnZmNfbWF5YmVf ZGVyZWZlcmVuY2VfdmFyKTogQXZvaWQgZCByZWZlcmVuY2luZyBhIG5vbnBvaW50ZXIuCiAg ICAKICAgIGdjYy90ZXN0c3VpdGUvQ2hhbmdlTG9nOgogICAgCiAgICAgICAgICAgICogZ2Zv cnRyYW4uZGcvY2xhc3NfcmVzdWx0XzEwLmY5MDogTmV3IHRlc3QuCgpkaWZmIC0tZ2l0IGEv Z2NjL2ZvcnRyYW4vdHJhbnMtZXhwci5jYyBiL2djYy9mb3J0cmFuL3RyYW5zLWV4cHIuY2MK aW5kZXggYzlkOWE5MTZjMjguLjcxZDAzNzEwMWQ0IDEwMDY0NAotLS0gYS9nY2MvZm9ydHJh bi90cmFucy1leHByLmNjCisrKyBiL2djYy9mb3J0cmFuL3RyYW5zLWV4cHIuY2MKQEAgLTI4 MDUsOSArMjgwNSw5IEBAIGNvbnZfcGFyZW50X2NvbXBvbmVudF9yZWZlcmVuY2VzIChnZmNf c2UgKiBzZSwgZ2ZjX3JlZiAqIHJlZikKICAgZHQgPSByZWYtPnUuYy5zeW07CiAgIGMgPSBy ZWYtPnUuYy5jb21wb25lbnQ7CiAKLSAgLyogUmV0dXJuIGlmIHRoZSBjb21wb25lbnQgaXMg aW4gdGhlIHBhcmVudCB0eXBlLiAgKi8KKyAgLyogUmV0dXJuIGlmIHRoZSBjb21wb25lbnQg aXMgaW4gdGhpcyB0eXBlLCBpLmUuIG5vdCBpbiB0aGUgcGFyZW50IHR5cGUuICAqLwogICBm b3IgKGNtcCA9IGR0LT5jb21wb25lbnRzOyBjbXA7IGNtcCA9IGNtcC0+bmV4dCkKLSAgICBp ZiAoc3RyY21wIChjLT5uYW1lLCBjbXAtPm5hbWUpID09IDApCisgICAgaWYgKGMgPT0gY21w KQogICAgICAgcmV0dXJuOwogCiAgIC8qIEJ1aWxkIGEgZ2ZjX3JlZiB0byByZWN1cnNpdmVs eSBjYWxsIGdmY19jb252X2NvbXBvbmVudF9yZWYuICAqLwpAQCAtMjg2Nyw2ICsyODY3LDgg QEAgdHJlZQogZ2ZjX21heWJlX2RlcmVmZXJlbmNlX3ZhciAoZ2ZjX3N5bWJvbCAqc3ltLCB0 cmVlIHZhciwgYm9vbCBkZXNjcmlwdG9yX29ubHlfcCwKIAkJCSAgIGJvb2wgaXNfY2xhc3Nh cnJheSkKIHsKKyAgaWYgKCFQT0lOVEVSX1RZUEVfUCAoVFJFRV9UWVBFICh2YXIpKSkKKyAg ICByZXR1cm4gdmFyOwogICBpZiAoaXNfQ0ZJX2Rlc2MgKHN5bSwgTlVMTCkpCiAgICAgcmV0 dXJuIGJ1aWxkX2ZvbGRfaW5kaXJlY3RfcmVmX2xvYyAoaW5wdXRfbG9jYXRpb24sIHZhcik7 CiAKZGlmZiAtLWdpdCBhL2djYy90ZXN0c3VpdGUvZ2ZvcnRyYW4uZGcvY2xhc3NfcmVzdWx0 XzEwLmY5MCBiL2djYy90ZXN0c3VpdGUvZ2ZvcnRyYW4uZGcvY2xhc3NfcmVzdWx0XzEwLmY5 MApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMC4uYTRkMjlhYjljMWQK LS0tIC9kZXYvbnVsbAorKysgYi9nY2MvdGVzdHN1aXRlL2dmb3J0cmFuLmRnL2NsYXNzX3Jl c3VsdF8xMC5mOTAKQEAgLTAsMCArMSw1MiBAQAorISB7IGRnLWRvIHJ1bn0KKworCishIFBS IGZvcnRyYW4vOTk1ODUKKworbW9kdWxlIG0yCisgIHR5cGUgdAorICAgICBjbGFzcygqKSwg cG9pbnRlciA6OiBiYXIoOikKKyAgZW5kIHR5cGUKKyAgdHlwZSB0MgorICAgICBjbGFzcyh0 KSwgYWxsb2NhdGFibGUgOjogbXkoOikKKyAgZW5kIHR5cGUgdDIKK2NvbnRhaW5zCisgIGZ1 bmN0aW9uIGYgKHgsIHkpIHJlc3VsdCh6KQorICAgIGNsYXNzKHQpIDo6IHgoOikKKyAgICBj bGFzcyh0KSA6OiB5KHNpemUoeCgxKSViYXIpKQorICAgIHR5cGUodCkgIDo6IHooc2l6ZSh4 KDEpJWJhcikpCisgIGVuZAorICBmdW5jdGlvbiBnICh4KSByZXN1bHQoeikKKyAgICBjbGFz cyh0KSA6OiB4KDopCisgICAgdHlwZSh0KSAgOjogeihzaXplKHgoMSklYmFyKSkKKyAgZW5k CisgIHN1YnJvdXRpbmUgcyAoKQorICAgIGNsYXNzKHQyKSwgYWxsb2NhdGFibGUgOjogYSg6 KSwgYig6KSwgYyg6KSwgZCg6KQorICAgIGNsYXNzKHQyKSwgcG9pbnRlciAgICAgOjogcCg6 KQorICAgIGMoMSklbXkgPSBmIChhKDEpJW15LCBiKDEpJW15KQorICAgIGQoMSklbXkgPSBn IChwKDEpJW15KQorICBlbmQKK2VuZAorCishIENvbnRyaWJ1dGVkIGJ5ICBHLiBTdGVpbm1l dHo6CishIFBSIGZvcnRyYW4vMTA0NDMwCisKK21vZHVsZSBtCisgICB0eXBlIHQKKyAgICAg IGludGVnZXIgOjogYQorICAgZW5kIHR5cGUKK2NvbnRhaW5zCisgICBmdW5jdGlvbiBmKHgp IHJlc3VsdCh6KQorICAgICAgY2xhc3ModCkgOjogeCg6KQorICAgICAgdHlwZSh0KSA6OiB6 KHNpemUoeCVhKSkKKyAgICAgIHolYSA9IDQyCisgICBlbmQKK2VuZAorcHJvZ3JhbSBwCisg ICB1c2UgbQorICAgY2xhc3ModCksIGFsbG9jYXRhYmxlIDo6IHkoOiksIHooOikKKyAgIGFs bG9jYXRlICh5KDMyKSkKKyAgIHogPSBmKHkpCisgICBpZiAoc2l6ZSh6KSAvPSAzMikgc3Rv cCAxCisgICBpZiAoYW55ICh6JWEgLz0gNDIpKSBzdG9wIDIKK2VuZAo= --------------TqsOHTwKiwhioUB8K1NFHB40--