233739 2008-08-02 19:40 0000 Make dev-lang/icc define __amd64 and __amd64__ preprocessor directives 2008-08-22 17:17:57 0000 1 1 1 Unclassified Gentoo Linux Ebuilds unspecified All Linux RESOLVED FIXED P2 normal --- 1 mattst88@gmail.com bicatali@gentoo.org mattst88@gmail.com 2008-08-02 19:40:32 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 mattst88@gmail.com 2008-08-02 19:40:56 0000 Created an attachment (id=162023) ebuild patch mattst88@gmail.com 2008-08-02 19:43:07 0000 Created an attachment (id=162024) add __amd64 preprocessor directives to stddef.h bicatali@gentoo.org 2008-08-22 17:17:57 0000 Now in cvs. Thanks much for tracking this! 162023 2008-08-02 19:40 0000 ebuild patch icc-ebuild.patch text/plain LS0tIGljYy0xMC4xLjAxNy5lYnVpbGQub3JpZwkyMDA4LTA4LTAyIDE1OjM5OjE2LjAwMDAwMDAw MCAtMDQwMAorKysgaWNjLTEwLjEuMDE3LmVidWlsZAkyMDA4LTA4LTAyIDE1OjM5OjMzLjAwMDAw MDAwMCAtMDQwMApAQCAtNTYsNiArNTYsNyBAQAogCQllaW5mbyAiRXh0cmFjdGluZyAkKGJhc2Vu YW1lICR7eH0pLi4uIgogCQlycG1fdW5wYWNrICIke1N9LyR7eH0iIHx8IGRpZSAicnBtX3VucGFj ayAke3h9IGZhaWxlZCIKIAlkb25lCisJZXBhdGNoICIke0ZJTEVTRElSfS9pY2MtYWRkLWFtZDY0 LXByZXByb2Nlc3Nvci1kaXJlY3RpdmVzLnBhdGNoIgogCiAJZWluZm8gIkZpeGluZyBwYXRocyBh bmQgdGFnZ2luZyIKIAljZCAiJHtTfSIvJHtJTlNUQUxMX0RJUn0vYmluCg== 162024 2008-08-02 19:43 0000 add __amd64 preprocessor directives to stddef.h icc-add-amd64-preprocessor-directives.patch text/plain LS0tIG9wdC9pbnRlbC9jY2UvMTAuMS4wMTcvaW5jbHVkZS9zdGRkZWYuaAkyMDA4LTA4LTAyIDE1 OjA2OjUxLjAwMDAwMDAwMCAtMDQwMAorKysgb3B0L2ludGVsL2NjZS8xMC4xLjAxNy9pbmNsdWRl L3N0ZGRlZi5oCTIwMDgtMDgtMDIgMTU6MDc6MDYuMDAwMDAwMDAwIC0wNDAwCkBAIC01LDYgKzUs MTMgQEAKICNkZWZpbmUgX1NURERFRgogKi8KIAorI2lmbmRlZiBfX2FtZDY0CisjZGVmaW5lIF9f YW1kNjQgX194ODZfNjQKKyNlbmRpZgorI2lmbmRlZiBfX2FtZDY0X18KKyNkZWZpbmUgX19hbWQ2 NF9fIF9feDg2XzY0X18KKyNlbmRpZgorCiAjaWZkZWYgX19RTlhfXyAvKiBKdXN0IHRha2UgUU5Y IG9uZSAqLwogI2luY2x1ZGVfbmV4dCA8c3RkZGVmLmg+CiAjZWxzZSAvKiBfX1FOWF9fICovCg==