-
Notifications
You must be signed in to change notification settings - Fork 205
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
With new clang based intel compilers (ifx, icx, icpx) we should use -march=native #4744
Comments
With 2024.0.0, |
Is there any reference material on this? I'm not sure we should trust the Intel (oneAPI) compilers to "do the right thing" on AMD-based systems when using |
Intel doesn't mention what happens for |
at most they say
but for me it was @bartoldeman telling me to just use it, and from my testing i can confirm that yes, that works. |
If one looks at the downstream source code, the behavior is basically the same from what I've found for LLVM:
This seems completely independent of the CPU vendor. Who knows though if they are using the same code in their oneAPI packages... |
PR to adopt We should ideally evaluate this a bit to see whether this is a good idea or may cause trouble. |
easybuild-framework/easybuild/toolchains/compiler/inteliccifort.py
Line 62 in b37f707
We should set this to
-march=native
but only when the new ifx, icx, icpx are used.This will be better for AMD cpus. I think @bartoldeman said that from 2025 onwards the old compilers are gone, so, we shouldn't use that old bad -x flags that are only supported on Intel cpus.
The text was updated successfully, but these errors were encountered: