On Tue, Jan 27, 2015 at 2:56 AM, Sandra Loosemore
<san...@codesourcery.com> wrote:
> On 01/20/2015 12:02 PM, H.J. Lu wrote:
>>
>> On Tue, Jan 20, 2015 at 10:51 AM, Eric Botcazou <ebotca...@adacore.com>
>> wrote:
>>>>
>>>> Ping?  Any thoughts?
>>>
>>>
>>> x86 for the family and x86-32/x86-64 for the 2 architectures?
>>>
>>
>> Works for me.
>
>
> [redirecting from gcc@ to gcc-patches@]
>
> OK, here is a patch that attempts to implement that convention.  I'd
> appreciate review from a target maintainer to check that I've correctly
> disambiguated places where "i386" was referring to both 32- and 64-bit
> variants vs 32-bit only.  I've left alone some instances of "i386" where it
> seemed appropriate to name a specific processor -- e.g. there are a bunch of
> examples in the inline asm section that are described as "i386 code".
>
> If this is OK to commit, I will follow it up with another patch to
> re-alphabetize the renamed sections ("i386 whatever" to "x86 whatever").
> Trying to do both the renaming and the shuffling in a single patch would
> have made it impossible to review the actual changes to content. When I was
> working on this I also realized that some of the x86-specific material in
> extend.texi really needs copy-editing; again, best to do that in a separate
> patch.

-@node i386 and x86-64 Options
-@subsection Intel 386 and AMD x86-64 Options
+@node x86 Options
+@subsection x86 Options
 @cindex i386 Options
-@cindex x86-64 Options
+@cindex x86 Options
+@cindex IA-32 Options
 @cindex Intel 386 Options
 @cindex AMD x86-64 Options

Let's go all the way and remove all but  "@cindex x86 Options".

-These @samp{-m} options are defined for the i386 and x86-64 family of
-computers:
+These @samp{-m} options are defined for the x86 family of computers,
+including both x86-32 (IA-32 and Intel 386) and AMD x86-64:

Also here. "... the x86 family of computers.". Without the "including ..." part.

-@node i386 and x86-64 Windows Options
-@subsection i386 and x86-64 Windows Options
-@cindex i386 and x86-64 Windows Options
+@node x86 Windows Options
+@subsection x86 Windows Options
+@cindex x86 Windows Options
+@cindex i386 Windows Options
+@cindex Intel 386 Windows Options
+@cindex AMD x86-64 Windows Options
+@cindex Windows Options for x86

IMO, all but "@cindex x86 Windows Options" should be removed.

Others LGTM.

Thanks,
Uros.

Reply via email to