First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 233739
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Sébastien Fabbro <bicatali@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Matt Turner <mattst88@gmail.com>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
icc-ebuild.patch ebuild patch patch Matt Turner 2008-08-02 19:40 0000 385 bytes Details | Diff
icc-add-amd64-preprocessor-directives.patch add __amd64 preprocessor directives to stddef.h patch Matt Turner 2008-08-02 19:43 0000 397 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 233739 depends on: Show dependency tree
Bug 233739 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

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) [edit]
ebuild patch

------- Comment #2 From Matt Turner 2008-08-02 19:43:07 0000 ------- 
Created an attachment (id=162024) [edit]
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!
First Last Prev Next    No search results available      Search page      Enter new bug