From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 51310 invoked by alias); 5 Feb 2020 16:38:33 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 51302 invoked by uid 89); 5 Feb 2020 16:38:32 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-4.0 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 spammy=replying, IRC, VLA, irc X-HELO: us-smtp-1.mimecast.com Received: from us-smtp-delivery-1.mimecast.com (HELO us-smtp-1.mimecast.com) (207.211.31.120) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 05 Feb 2020 16:38:31 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1580920709; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=eRu4RlTgNQ05iHuXJwGHCIV+J6QRjYQlD6/fTdV3fZU=; b=A/5I8lE/jxi1sOtgG9LJeQiQTCl2SUah8rLxGdFV+NBSh7g2M/hcyFJ7HkxBuSQpyMlDrh VU4+xtOFGPWhFJFpXXyS5fGXN7GvkUzIK7Me9tACHi4O60LtKV7CtHAE4HEKuaiJAx8Ba+ KIjbXNvJne2nycUBLlWzV8MTc4zQL7M= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-201-9rZT3s-7PmeV-e6y7ENr6w-1; Wed, 05 Feb 2020 11:38:11 -0500 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 1D33A1137846; Wed, 5 Feb 2020 16:38:10 +0000 (UTC) Received: from localhost (unused-10-15-17-196.yyz.redhat.com [10.15.17.196]) by smtp.corp.redhat.com (Postfix) with ESMTP id BB059790D4; Wed, 5 Feb 2020 16:38:09 +0000 (UTC) From: Sergio Durigan Junior To: Andrew Burgess Cc: gdb-patches@sourceware.org, Joel Brobecker Subject: Re: [PATCH] Add gdb.fortran/vla-stride.exp and report a bug References: <87sgkivgx2.fsf_-_@redhat.com> <20200118230433.GC3865@embecosm.com> Date: Wed, 05 Feb 2020 16:38:00 -0000 In-Reply-To: <20200118230433.GC3865@embecosm.com> (Andrew Burgess's message of "Sat, 18 Jan 2020 23:04:33 +0000") Message-ID: <87eev9kn0u.fsf@redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2020-02/txt/msg00102.txt.bz2 On Saturday, January 18 2020, Andrew Burgess wrote: > * Sergio Durigan Junior [2020-01-13 22:46:33 -0500]: > >> On Thursday, November 14 2019, Andrew Burgess wrote: >>=20 >> > Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/627 >> > ...................................................................... >> > >> > gdb/fortran: array stride support >> > [...] >>=20 >> Hey Andrew, >>=20 >> I found a problem with this patch, and I'd like to know if you've >> noticed this as well. I first encountered the problem while doing >> downstream work on Fedora GDB for Fedora Rawhide; as you are probably >> aware, we carry *a lot* of local Fortran VLA patches on Fedora GDB (if >> you're not aware about this, feel free to get in touch with me and I'll >> be more than happy to explain the situation to you). However, I am able >> to reproduce the problem on upstream GDB as well. >>=20 >> On Fedora GDB, we carry a testcase called gdb.fortran/vla-stride.exp. I= 'm >> attaching it to this message. One of its tests fails with: >>=20 >> (gdb) print pvla >> Cannot access memory at address 0x426000 >> FAIL: gdb.fortran/vla-stride.exp: print single-element > > Sergio, > > First, apologies for not replying sooner, I completely missed this > mail. My bad! That's totally fine -- I also took too long to reply. > Thanks for the bug report. Yes I'm aware that Fedora carries some > Fortran patches, its on my (ever growing) todo list that I should take > a look at them one day. It kind-of sucks that my top of tree GDB is > sometimes not as good as my slightly older distro-installed GDB! Hah, yeah :-). It'd be great to have your help with the patches; let me know when you can dedicate some time, and we can coordinate efforts. > I put together a patch for this issue (see below) it passes your test > case on my machine (with no other regressions), but it would be neat > if you could confirm it resolves the issue for you. Thanks a lot. I've just confirmed that the patch indeed fixes the issue. > One question - I included your testcase in this patch, I just wanted > to check that this is OK for upstream (w.r.t. copyright assignment, etc)? So, as we discussed on IRC, I don't have the exact authorship information for the testcase, but I think it was written by Bernhard Heckel . I'm almost sure that this was contributed by Intel, and if that's the case, then they have copyright assignment on file with us, so it should be fine to push the patch as is. I'll let you know if I find more info on this, but apparently (based on our IRC talk) you're going to write a new test anyway, so please don't wait for me. Cheers, --=20 Sergio GPG key ID: 237A 54B1 0287 28BF 00EF 31F4 D0EB 7628 65FC 5E36 Please send encrypted e-mail if possible http://sergiodj.net/