This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [wwwdocs] index.html further news tweaks


Richard Kenner wrote:

>> Gerald Pfeifer wrote:

I'm not a native speaker, but

In the x86-64 compiler, @option{-malign-double} is the default choice.

somehow doesn't flow well for me. Perhaps this is just me, or perhaps something like might be better?

On x86-64 @option{-malign-double} is enabled by default.
I think your version is slightly better than Toon's, but I think both
are grammatical and would be understood by native English speakers.

Richard Kenner replied:


Well, except for the missing comma in the second one.

I prefer the second too because "choice" to me tends to imply more
than just enabled or disabled, as if there were more than two choices.

I committed the following (see attached patch).


Is this useful for the 4.1 branch, too ? It isn't a regression, but for documentation we might want to have more relaxed rules ...

Cheers,

--
Toon Moene - e-mail: toon@moene.indiv.nluug.nl - phone: +31 346 214290
Saturnushof 14, 3738 XG  Maartensdijk, The Netherlands
A maintainer of GNU Fortran 95: http://gcc.gnu.org/fortran/
Gfortran At Work: http://mysite.verizon.net/serveall/moene.pdf
2006-07-17  Toon Moene  <toon@moene.indiv.nluug.nl>

	* invoke.texi: Remove mention of f77; Reformat table for
	inline parameter options correctly; mention that -malign-double
	is default on x86-64 targets.

Index: invoke.texi
===================================================================
--- invoke.texi	(revision 115432)
+++ invoke.texi	(working copy)
@@ -943,7 +943,6 @@
 objective-c++ objective-c++-header objective-c++-cpp-output
 assembler  assembler-with-cpp
 ada
-f77  f77-cpp-input
 f95  f95-cpp-input
 java
 treelang
@@ -4652,14 +4651,14 @@
 as follows:
 
 @table @gcctabopt
- @item max-inline-insns-single
-  is set to @var{n}/2.
- @item max-inline-insns-auto
-  is set to @var{n}/2.
- @item min-inline-insns
-  is set to 130 or @var{n}/4, whichever is smaller.
- @item max-inline-insns-rtl
-  is set to @var{n}.
+@item max-inline-insns-single
+ is set to @var{n}/2.
+@item max-inline-insns-auto
+ is set to @var{n}/2.
+@item min-inline-insns
+ is set to 130 or @var{n}/4, whichever is smaller.
+@item max-inline-insns-rtl
+ is set to @var{n}.
 @end table
 
 See below for a documentation of the individual
@@ -9374,6 +9373,8 @@
 produce code that runs somewhat faster on a @samp{Pentium} at the
 expense of more memory.
 
+On x86-64, @option{-malign-double} is enabled by default.
+
 @strong{Warning:} if you use the @option{-malign-double} switch,
 structures containing the above types will be aligned differently than
 the published application binary interface specifications for the 386
@@ -13081,7 +13082,7 @@
 @opindex fbounds-check
 For front-ends that support it, generate additional code to check that
 indices used to access arrays are within the declared range.  This is
-currently only supported by the Java and Fortran 77 front-ends, where
+currently only supported by the Java and Fortran front-ends, where
 this option defaults to true and false respectively.
 
 @item -ftrapv

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]