On Thu, Feb 23, 2017 at 11:13 PM, Martin Sebor <mse...@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 --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
