-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Some fixes for building on windows + cuda #1614
Conversation
Windows is not POSIX-compliant and does not provide clock_gettime() Also, the timespec type from time.h is C11, compare which MSVC has not supported for the longest time (only starting from 14.28 / Visual Studio 16.8), cf. https://en.cppreference.com/w/c/chrono/timespec Change to an implementation based on C++11 std-lib, see https://en.cppreference.com/w/cpp/chrono
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wickedfoo has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Thanks for the diff! I've imported it internally and take a look on Monday to make sure it passed our internal build and then will land it. |
Cool, happy to hear it! Together with d2c04d0 and the commits here, I managed to get to the linker stage in conda-forge/faiss-split-feedstock#19, but there it fails with
which I haven't been able to resolve. |
@wickedfoo merged this pull request in 3d3d539. |
Upstreaming some work towards #1586 from conda-forge/faiss-split-feedstock#19; complements #1610