Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Intel compiler to CPU Makefiles (And .gitignore) #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ZuluPro
Copy link

@ZuluPro ZuluPro commented Apr 20, 2018

This does not change the previous behavior. If a user does only make, it'll use g++.
But user can now do make icpc.

Maybe you want me to add this to another makefiles ?
Or docs later.

@ZuluPro
Copy link
Author

ZuluPro commented Apr 24, 2018

@tristanvdb @cavazos
Any news ?

@ZuluPro
Copy link
Author

ZuluPro commented May 5, 2018

@tristanvdb @cavazos Any news ? Something wrong on my code ?

@ZuluPro
Copy link
Author

ZuluPro commented May 20, 2018

@tristanvdb @cavazos Any news ?

1 similar comment
@ZuluPro
Copy link
Author

ZuluPro commented Jun 4, 2018

@tristanvdb @cavazos Any news ?

@ZuluPro
Copy link
Author

ZuluPro commented Jun 9, 2018

Still no news...............

@VincentSC
Copy link
Collaborator

We've asked to take over the project, so the code could be tuned and optimized for new and recent architectures. We're waiting for an email response. The unaltered fork is currently at https://github.com/StreamHPC/FinanceBench

Your MR could better use the CXX variable to make it more generic. Then add parameter -xHost only for icpc. Then it would use the default compiler, unless overridden.
Project can better be ported to cmake to make it more flexible, including using another compiler than gcc, including clang.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants