-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
[vtk-m:x64-windows] build failure, no supported version of OpenMP #13014
Comments
./vcpkg install vtk-m[cuda,double,mpi, |
It seem the feature 'omp' doesn't support well in vtk-m, see https://gitlab.kitware.com/vtk/vtk-m/-/issues/139 I think we need to remove this feature. |
The CUDA feature also broken, I will disable it as well. Many CUDA issues Active in https://gitlab.kitware.com/vtk/vtk-m/-/issues?scope=all&utf8=%E2%9C%93&state=opened&search=CUDA, consider to test it when https://gitlab.kitware.com/vtk/vtk-m/-/issues/363 resolved, if everything test successfully, we can enable this feature. Confirmed ./vcpkg install vtk-m[double,mpi,tbb]:x64-windows installed successfully. |
CUDA probably requires to put the CUDA compiler nvcc actually on path like VTK does. |
And yeah the openMP issue is due to VS not supporting openMP well enough. Removing the feature seems overkill since I am free to configure another compiler on Windows which supports openmp 4.0. So testing this on Linux is mandatory before removing any features! |
icl intel support openmp 4+ ver, clang support 4+ ver ... windows msvc 3- ver experimental ;(( |
@Neumann-A, thanks for pointing out this! I test the omp feature on linux, it looks installed successfully, so removing the feature is not wise. Very appreciate for your information! I think all features need to be test on windows and linux when they has been added, if it doesn't support on specific arch or platform, we should have related information or commnents in future. That will save time for users. |
hmm... cl -openmp:experimental = 4 ? //test it someone |
@PhoebeHui : CUDA is problematic but not so problematic. In most cases, it's more about the installation of CUDA (the correct version for GPU, setting CUDA_PATH for CMake or finding the supported version host compiler when built on Linux). There is limited check for this in CMake. So I think so far this should not be a priority. The MPI support (for vtk-m and vtk) needs more attention. |
Verified vtk-m features with latest vcpkg source with x64-windows, vtk-m[double, mpi, tbb]:x64-windows test pass, cuda and omp failed as expected. Feature mpi passed on x64-windows and x64-linux. |
Host Environment
To Reproduce
Steps to reproduce the behavior:
./vcpkg install vtk-m[cuda,double,mpi,omp,tbb]:x64-windows
Failure logs
The text was updated successfully, but these errors were encountered: