Bug 233739 - Make dev-lang/icc define __amd64 and __amd64__ preprocessor directives
Bug#: 233739 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: normal Priority: P2
Resolution: FIXED Assigned To: bicatali@gentoo.org Reported By: mattst88@gmail.com
Component: Ebuilds
URL: 
Summary: Make dev-lang/icc define __amd64 and __amd64__ preprocessor directives
Keywords:  
Status Whiteboard: 
Opened: 2008-08-02 19:40 0000
Description:   Opened: 2008-08-02 19:40 0000
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 From Matt Turner 2008-08-02 19:40:56 0000 -------
Created an attachment (id=162023) [details]
ebuild patch

------- Comment #2 From Matt Turner 2008-08-02 19:43:07 0000 -------
Created an attachment (id=162024) [details]
add __amd64 preprocessor directives to stddef.h

------- Comment #3 From Sébastien Fabbro 2008-08-22 17:17:57 0000 -------
Now in cvs. Thanks much for tracking this!