Created attachment 15341 [details] [review] Add checks for __x86_64__ everytime we check for __amd64__ It seems Intel C Compiler neglects to define __amd64__, __amd64, or amd64, but *does* define __x86_64__. Politics anyone? This patch adds checks for __x86_64__ anytime code checks for one of the three previous. Btw, gcc defines all but 'amd64'.
Created attachment 15342 [details] [review] Found one more. Same as previous patch + one more I forgot to add.
This is an extremely trivial bug. Can someone take 2 minutes and push this patch through?
> --- Comment #2 from Matt Turner <mattst88@gmail.com> 2008-04-12 11:29:25 PST --- > This is an extremely trivial bug. Can someone take 2 minutes and push this > patch through? > commit 1fa4de80fcfc697b5e5879cc351fb3e9dbf6acbe Author: Matt Turner <mattst88@gmail.com> Date: Sat Apr 12 20:39:18 2008 +0200 Check for __x86_64__ when we check for __amd64__ It seems Intel C Compiler neglects to define __amd64__, __amd64, or amd64, but *does* define __x86_64__.
Created attachment 15913 [details] [review] Remove all amd64 define checks in favor of __x86_64__ In at least two files, the previous patch duplicated checks for __x86_64__. This patch fixes this, and also replaces all checks for amd64, __amd64, and __amd64 with checks for __x86_64__. Discussed on #xorg-devel yesterday.
Created attachment 16514 [details] [review] Fix duplicate checks introduced by original patch
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.