-
Notifications
You must be signed in to change notification settings - Fork 256
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
CMake OpenMP target incompatible with Clang using MSVC and GNU CLI #750
Comments
That is probably due to the same issue that is mentioned here: If there is no open bug report about that for CMake yet, it might make sense to create one... |
A work-around for this issue until it is fixed in upstream CMake is to use |
Is this resolved now, by using |
Using It might be best to keep this open until this is fixed in case someone else runs into the same issue. |
See #760 for a reproducer in CI. |
I opened a merge request for CMake proposing the change from #760: |
The change has been accepted upstream and should be part of CMake 3.29 when it will be released (unless it will be reverted for some reason before the release). I don't know how often CMake is updated in MS Visual Studio. It is at |
CMake 3.29 (for which this should be fixed) has just been released: I don't know how long it will take until that version finds its way into MS Visual Studio. |
In the closed issue #309, @chengts95 noted in the 30th comment:
Opening a separate issue so that this won't get forgotten.
The build errors like the following:
The error probably occurs because the OpenMP libraries aren't set for some reason by
FindOpenMP.cmake
:IIUC, they should be set to the same as when building with
clang-cl
:The text was updated successfully, but these errors were encountered: