SFML community forums

General => Feature requests => Topic started by: dnob700 on September 05, 2007, 01:10:26 am

Title: Simple modification for SFML to work with Intel C++ compiler
Post by: dnob700 on September 05, 2007, 01:10:26 am

Hello,

I'm using Ubuntu linux with a Core 2 Duo processor, i.e. an AMD64 platform.

I compiled successfully SFML with gcc. but in order to use it with icc (the intel compiler), the line 70 of the Config.hpp header should be :
#elif defined(__amd64__) || defined(__x86_64) || defined(__x86_64__)

as the platform is named x86_64 for the intel compiler and not amd64. I did not try to build the library with icc, but I suppose that such modifications are also needed elsewhere (If necessary I can try to build it with this compiler).

thank you for your great work.

Title: Simple modification for SFML to work with Intel C++ compiler
Post by: Laurent on September 05, 2007, 08:54:57 am

Thank you very much, I've added your fix.

However it should compile fine (there is no architecture specific code), so if you want you can try to compile it without any other modification.

Title: Simple modification for SFML to work with Intel C++ compiler
Post by: CareBear on September 05, 2007, 04:35:58 pm

Adding "|| defined(_WIN64)" on line 71 would be fine too, for 64bit windows.
Maybe look at the patch i posted yesterday:

http://sourceforge.net/tracker/index.php?func=detail&aid=1787741&group_id=188964&atid=927429

Title: Simple modification for SFML to work with Intel C++ compiler
Post by: Laurent on September 05, 2007, 05:32:21 pm

Quote

Adding "|| defined(_WIN64)" on line 71 would be fine too, for 64bit windows.

But I need to make a difference between 64 bits architectures (AMD64, IA64, ...) not between 64 bits operating systems.

Title: Simple modification for SFML to work with Intel C++ compiler
Post by: CareBear on September 05, 2007, 06:48:04 pm

Ouch,... yes :?
My patch would add "defined(__x86_64__)" as well, but right, the _WIN64 should not be there. However, it works now with the 64bit compiler of the Platform SDK. So it would be better if you close/delete this patch, because its needless now.

Title: Simple modification for SFML to work with Intel C++ compiler
Post by: Laurent on September 05, 2007, 07:04:21 pm

Yep ;)