Welcome, Guest. Please login or register. Did you miss your activation email?

Author Topic: Simple modification for SFML to work with Intel C++ compiler  (Read 29152 times)

0 Members and 1 Guest are viewing this topic.

dnob700

  • Newbie
  • *
  • Posts: 8
    • View Profile
Simple modification for SFML to work with Intel C++ compiler
« 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.

Laurent

  • Administrator
  • Hero Member
  • *****
  • Posts: 32498
    • View Profile
    • SFML's website
    • Email
Simple modification for SFML to work with Intel C++ compiler
« Reply #1 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.
Laurent Gomila - SFML developer

CareBear

  • Newbie
  • *
  • Posts: 13
    • View Profile
Simple modification for SFML to work with Intel C++ compiler
« Reply #2 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

Laurent

  • Administrator
  • Hero Member
  • *****
  • Posts: 32498
    • View Profile
    • SFML's website
    • Email
Simple modification for SFML to work with Intel C++ compiler
« Reply #3 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.
Laurent Gomila - SFML developer

CareBear

  • Newbie
  • *
  • Posts: 13
    • View Profile
Simple modification for SFML to work with Intel C++ compiler
« Reply #4 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.

Laurent

  • Administrator
  • Hero Member
  • *****
  • Posts: 32498
    • View Profile
    • SFML's website
    • Email
Simple modification for SFML to work with Intel C++ compiler
« Reply #5 on: September 05, 2007, 07:04:21 pm »
Yep ;)
Laurent Gomila - SFML developer