Bug 233739 – Make dev-lang/icc define __amd64 and __amd64__ preprocessor directives

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Matt Turner 2008-08-02 19:40:32 UTC]

As per the AMD64 ABI, compliant compilers must define __amd64, __amd64__, __x86_64, and __x86_64__ preprocessor directives[1]. icc does not define __amd64 or __amd64__, and Intel has stated that it has no plans to change this[2]. Many software projects check for __amd64__ only (ie, don't check for __x86_64__), for instance, the xserver. The developers have also expressed no interest in checking for __x86_64__ for the sole reason of helping Intel, for instance, Adam Jackson, Xorg release manager.[3]

[1] http://www.x86-64.org/documentation/abi.pdf (See Chapter 7)
[2] http://softwarecommunity.intel.com/isn/Community/en-US/forums/post/30251206.aspx
[3] http://cgit.freedesktop.org/xorg/xserver/commit/?id=9719354ae04a41e9834ac9ba8fd9e895092ba4f1

This patch adds __amd64 and __amd64__ defines to icc's stddef.h.

Please add and bump to 10.1.017-r1.

Reproducible: Always

[Comment 1 Matt Turner 2008-08-02 19:40:56 UTC]

Created attachment 162023 [details, diff]
ebuild patch

[Comment 2 Matt Turner 2008-08-02 19:43:07 UTC]

Created attachment 162024 [details, diff]
add __amd64 preprocessor directives to stddef.h

[Comment 3 Sébastien Fabbro (RETIRED) 2008-08-22 17:17:57 UTC]

Now in cvs. Thanks much for tracking this!