Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: "Ulrich Weigand" <uweigand@de.ibm.com>
Cc: brobecker@adacore.com, gdb-patches@sourceware.org
Subject: Re: Doc update to mention MPFR (Re: [RFC][00/19] Target FP: Precise target floating-point emulation)
Date: Mon, 20 Nov 2017 17:24:00 -0000	[thread overview]
Message-ID: <83d14ceue3.fsf@gnu.org> (raw)
In-Reply-To: <20171120134413.0F07CD802C3@oc3748833570.ibm.com>	(uweigand@de.ibm.com)

> Date: Mon, 20 Nov 2017 14:44:12 +0100 (CET)
> From: "Ulrich Weigand" <uweigand@de.ibm.com>
> Cc: brobecker@adacore.com, gdb-patches@sourceware.org
> 
> --- binutils-gdb.orig/gdb/NEWS
> +++ binutils-gdb/gdb/NEWS
> @@ -3,6 +3,10 @@
>  
>  *** Changes since GDB 8.0
>  
> +* GDB now uses the GNU MPFR library, if available, to emulate target
> +  floating-point arithmetic during expression evaluation when the target
> +  uses different floating-point formats than the host.
> +

This part is OK.

> +   GDB uses GNU MPFR, a library for multiple-precision floating-point
> +computation with correct rounding, to emulate target floating-point
> +arithmetic during expression evaluation when the target uses different
> +floating-point formats than the host.  MPFR will be linked in if it is
> +available at build time.  At least version 3.1 of GNU MPFR is required.
> +If GNU MPFR it is not available, GDB will fall back to using host
> +floating-point arithmetic.  The latest version of GNU MPFR should be
> +available from `http://www.mpfr.org´.
                                      ^
Some non-ASCII character sneaked in here.

> +`--with-mpfr'
> +     Build GDB with the GNU MPFR library.  (Done by default if
> +     GNU MPFR is installed and found at configure time.)  This library
> +     is used to emulate target floating-point arithmetic during expression
> +     evaluation when the target uses different floating-point formats than
> +     the host. At least version 3.1 of GNU MPFR is required.  If GNU MPFR

I would drop the reference to minimum supported MPFR version here: it
tends to become outdated as time goes by, and I believe the configure
script includes the necessary tests for what we need from MPFR, and
will reject versions that don't fit the bill.

(It is okay to have the version mentioned in NEWS, as that is a
one-time announcement.)

Also, please make sure there are 2 spaces between sentences.

> +@item MPFR
> +@anchor{MPFR}
> +@value{GDBN} can use the GNU MPFR multiple-precision floating-point
> +library.  This library may be included with your operating system
> +distribution; if it is not, you can get the latest version from
> +@url{http://www.mpfr.org}.  At least version 3.1 of GNU MPFR is
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Same here.

> +formats than the host.  If GNU MPFR it is not available, GDB will
                                                            ^^^
@value{GDBN}

The changes are okay with those gotchas fixed.

Thanks.


  reply	other threads:[~2017-11-20 17:24 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-05 18:20 [RFC][00/19] Target FP: Precise target floating-point emulation Ulrich Weigand
2017-09-05 18:25 ` Eli Zaretskii
2017-09-06 18:01   ` Ulrich Weigand
2017-09-06 18:37     ` Eli Zaretskii
2017-09-06 21:04       ` Joel Brobecker
2017-11-16 19:06         ` Ulrich Weigand
2017-11-16 19:26           ` Eli Zaretskii
2017-11-20 13:44             ` Doc update to mention MPFR (Re: [RFC][00/19] Target FP: Precise target floating-point emulation) Ulrich Weigand
2017-11-20 17:24               ` Eli Zaretskii [this message]
2017-11-20 17:39                 ` Ulrich Weigand
2017-11-20 18:25                   ` Eli Zaretskii

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=83d14ceue3.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=uweigand@de.ibm.com \
    /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