This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ping] Re: proper name of i386/x86-64/etc targets
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Sandra Loosemore <sandra at codesourcery dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, Eric Botcazou <ebotcazou at adacore dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Henderson <rth at redhat dot com>, Jan Hubicka <hubicka at ucw dot cz>
- Date: Tue, 27 Jan 2015 18:17:05 +0100
- Subject: Re: [ping] Re: proper name of i386/x86-64/etc targets
- Authentication-results: sourceware.org; auth=none
- References: <54A5E698 dot 60702 at codesourcery dot com> <54BDBCF0 dot 9050801 at codesourcery dot com> <1470230 dot JDgFJo2Qyv at polaris> <CAMe9rOonSQrGjr3qFohdp6_x--7wPAu=_zr345oKt9MOxYmi1Q at mail dot gmail dot com> <54C6F04A dot 2000904 at codesourcery dot com>
On Tue, Jan 27, 2015 at 2:56 AM, Sandra Loosemore
<sandra@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 <ebotcazou@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.