-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
fmt fail to compile with icc 15 (intel compiler) #822
Comments
Unfortunately I don't have an access to icc to debug this. Does changing #ifdef __has_include to #ifdef __has_include && (!defined(__INTEL_COMPILER) || __INTEL_COMPILER >= 1600) in https://github.com/fmtlib/fmt/blob/master/include/fmt/core.h#L27 fix the issue? I also recommend reporting to ICC developers since it looks like a compiler bug. |
hi @vitaut what you gave didn't work but this did
but now i have this issue
if i compile with tests disable it works, so it's good enough for me. |
BTW it compiles well (without tests) with ICC V15 and V17 |
The first issue is fixed (or rather worked around since it's a compiler bug) in ce19309. |
Regarding the second issue (which is even more obviously a compiler bug), do the tests compile if you replace if (HAVE_FNO_EXCEPTIONS_FLAG) with if (FALSE) in https://github.com/fmtlib/fmt/blob/master/test/CMakeLists.txt#L127 ? |
now i get this error
|
To get preprocessed file, first run make with
Then copy the compile command that gives an error, replace And finally run it:
|
I've implemented the proposed workarounds in a11eb3a (not merged yet). |
Merged workarounds for known icc bugs so closing for now. |
The text was updated successfully, but these errors were encountered: