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

[Martin Sebor <msebor at gmail dot com>]

On 02/23/2017 12:01 PM, Uros Bizjak wrote:
> On Thu, Feb 23, 2017 at 7:07 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"

Martin