Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Jan Kratochvil <jan.kratochvil@redhat.com>
To: Markus Deuling <deuling@de.ibm.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [6/6] Fortran dynamic arrays #2: Fortran array testcase
Date: Tue, 11 Mar 2008 09:39:00 -0000	[thread overview]
Message-ID: <20080311093849.GA16853@host0.dyn.jankratochvil.net> (raw)
In-Reply-To: <47D6478B.9000605@de.ibm.com>

On Tue, 11 Mar 2008 09:49:15 +0100, Markus Deuling wrote:
...
> I also saw cases when gfortran emitted wrong or incomplete debug data but it gets
> better with 4.3.

The Fortran dynamic arrays debug info fix got committed for 4.3 by Jakub Jelink:
	http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22244


> Just a small comment to your testcase:
>
> +if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug f77 quiet}] != "" } {
> +    untested "Couldn't compile ${srcfile}"
> +    return -1
> +}
>
> I'm not able to compile your .f90 testcase with f77. What I did is to invoke gfortran instead, hoping
> 4.3 gets better :-)
>
> +if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug compiler=gfortran quiet}] != "" } {
> +    untested "Couldn't compile ${srcfile}"
> +    return -1
> +}

While it is correct it is in fact more a general problem of all of the upstream
Fortran testcases as AFAIK `g77' binary/link is just no longer present in GCC.
Using now a workaround in:
	http://cvs.fedora.redhat.com/viewcvs/*checkout*/rpms/gdb/devel/gdb.spec
	cd $the_build_tree_root
	# g77 executable is no longer present in Fedora gcc-4.x+.
	g77="`which gfortran 2>/dev/null || true`"
	test -z "$g77" || ln -s "$g77" ./g77

Also the whole Fortran dynamic arrays patchset no longer applies well so it was
forward-ported for 6.8pre in between.  But only in its merged form:
	http://cvs.fedora.redhat.com/viewcvs/*checkout*/rpms/gdb/devel/gdb-6.8-bz377541-fortran-dynamic-arrays.patch


Regards,
Jan


      reply	other threads:[~2008-03-11  9:39 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-23 23:44 Jan Kratochvil
2008-03-11  8:49 ` Markus Deuling
2008-03-11  9:39   ` Jan Kratochvil [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080311093849.GA16853@host0.dyn.jankratochvil.net \
    --to=jan.kratochvil@redhat.com \
    --cc=deuling@de.ibm.com \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox