diff mbox

[doc] : Mention that -mfpmath=sse is the default on 32bit x86 w/ SSE2 and -ffast-math

Message ID CAFULd4aPLRiuXN7pBgdefoRa-HsTT6+43qyFXUYXadjZYnce-Q@mail.gmail.com
State New
Headers show

Commit Message

Uros Bizjak Feb. 24, 2017, 6:20 p.m. UTC 
On Thu, Feb 23, 2017 at 11:13 PM, Martin Sebor <msebor@gmail.com> wrote:

>>> A minor grammatical nit:
>>>
>>>   +This is the default choice for most of x86-32 targets.
>>>
>>> "for most x86-32 targets" is correct unless the targets are some
>>> specific subset, in which case "most of the [previously mentioned]
>>> x86-32 targets" would work.
>>
>>
>> Maybe we can say "This is the default choice for non-Darwin x86-32
>> targets." here?
>>
>> And further extend:
>> "This is the default choice for the x86-64 compiler, Darwin x86-32
>> targets, and the default choice for x86-32 targets with SSE2
>> instruction set when @option{-ffast-math} is enabled."
>
>
> The phrasing looks good to me.  There should be an article before
> SSE2, i.e., "targets with the SSE2 instruction set"

Attached is the committed patch.

Thanks,
Uros.
diff mbox

Patch

diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 6e5fa56..3322281 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -24591,7 +24591,7 @@  The temporary results are computed in 80-bit precision instead of the precision
 specified by the type, resulting in slightly different results compared to most
 of other chips.  See @option{-ffloat-store} for more detailed description.
 
-This is the default choice for x86-32 targets.
+This is the default choice for non-Darwin x86-32 targets.
 
 @item sse
 Use scalar floating-point instructions present in the SSE instruction set.
@@ -24611,7 +24611,9 @@  The resulting code should be considerably faster in the majority of cases and av
 the numerical instability problems of 387 code, but may break some existing
 code that expects temporaries to be 80 bits.
 
-This is the default choice for the x86-64 compiler.
+This is the default choice for the x86-64 compiler, Darwin x86-32 targets,
+and the default choice for x86-32 targets with the SSE2 instruction set
+when @option{-ffast-math} is enabled.
 
 @item sse,387
 @itemx sse+387