RE: [ping][patch] Fixed-point patch 4/10

["Fu, Chao-Ying" <fu at mips dot com>]

Mark Mitchell wrote:
> 
> >   Could you review the non-C-front-end part of this patch?
> > http://gcc.gnu.org/ml/gcc-patches/2007-08/msg01070.html
> > 
> >   This includes a new target hook and a config option to enable the
> > fixed-point extension.  Thanks a lot!
> 
> +Enable (or disable) support for the C fixed-point arithmetic
> 
> Remove "the".  End with a period.
> 
> > +This is enabled by default only on MIPS.  Other systems 
> may also support it,
> > +but require the user to specifically enable it.
> > +extension.
> 
> Future-proof this by saying:
> 
> "This option is enabled by default for some targets (such as 
> MIPS) which
> have hardware-support for fixed-point operations.  On other 
> targets, you
> may enable this option manually."
> 
> OK with those changes.
> 

  Thanks a lot!  Here is the diff file for install.texi.
  I plan to check in this patch (4/10) next Monday.

Regards,
Chao-ying

Index: install.texi
===================================================================
--- install.texi        (revision 127589)
+++ install.texi        (working copy)
@@ -1317,6 +1317,13 @@
 default on i386 and x86_64 systems, and the @samp{dpd} (densely packed
 decimal) format is default on PowerPC systems.

+@item --enable-fixed-point
+@itemx --disable-fixed-point
+Enable (or disable) support for C fixed-point arithmetic.
+This option is enabled by default for some targets (such as MIPS) which
+have hardware-support for fixed-point operations.  On other targets, you
+may enable this option manually.
+
 @item --with-long-double-128
 Specify if @code{long double} type should be 128-bit by default on selected
 GNU/Linux architectures.  If using @code{--without-long-double-128},