-
Notifications
You must be signed in to change notification settings - Fork 7.1k
[libdivide] Add new port #8320
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
[libdivide] Add new port #8320
Conversation
Failed on x86-windows due to this:
Failed on x64-uwp due to this:
Can someone help solve this issue or give any suggestions? |
If you are able to reproduce the x86 issue then you should report it https://docs.microsoft.com/en-us/cpp/overview/how-to-report-a-problem-with-the-visual-cpp-toolset?view=vs-2019. |
Thanks Phil for the review, I will help follow up. |
Hi, I am the current maintainer of libdivide. I have now fixed the The issue below however seems to be a x86 MSVC (32-bit) bug that is caused by libdivide's AVX2 code.
Note that this bug does not render libdivide useless since vector instructions are disabled by default and the user has to explicitly define Since I don't know how long it will take until this bug gets fixed in MSVC I have added an option to libdivide's CMake build system which allows to turn off CPU vector instructions for compiling the test programs. So the bug above can now be avoided using e.g.: cmake .. -G "Visual Studio 15 2017" -DENABLE_VECTOR_EXTENSIONS=OFF
cmake --build . --config Release I have also added |
@kimwalisch thanks a lot for your detail description about this issue. We will update this port if you have a new release. |
I have now released libdivide-3.0 on GitHub. Fingers crossed that it will work well with vcpkg... |
@kimwalisch thanks for your instant update . I have updated |
Thanks @NancyLi1013 for the port addition and thanks @kimwalisch for taking interest in helping us get libdivide in vcpkg! |
This port is a header-only library.
Related issue #8264.