-
Notifications
You must be signed in to change notification settings - Fork 87
Rudimentary CMake support #29
base: master
Are you sure you want to change the base?
Conversation
Merge c0rp3n/fastnoise-simd to torss/FastNoiseSIMD
Thank you for your work on this, I'm still figuring out cmake. I'm using it in FastNoise2 but I haven't hooked up any config or CI support. Could you give a run down on what platform/config the CI builds cover and what setup will they need. Also what are the cmake options and setup steps, I will need to add some info to the wiki. Thanks again for your help with this. |
@torss Why has the open simplex change been included now? |
This was added because setting these flags for MSVC did cause a runtime library mismatch in a project using FastNoiseSIMD via add_subdirectory.
Oops, sorry, I completely forgot about this old PR from 2018. ;) I'm currently only using my fork in a private test project. I just merged @c0rp3n's fork because I noticed that it apparently extends CMake support (although I personally don't need that) and @KdotJPG's fork since I wanted to try out his BTW nice to see that you are working on a successor, didn't know about |
So with my fork I added support for The config works by just calling And for buiding it would just be mkdir build
cd build
cmake -DFN_COMPILE_AVX2=ON -DFN_COMPILE_AVX512=OFF .. && make and then to install sudo make install With windows it differs slightly need to pass you need to set where you want to install it to on configure so cmake -DCMAKE_INSTALL_PREFIX="E:/deps" .. There will then be an install target added to the VS project which you can then use to install the library. Edit: |
Note that this changes
FastNoiseSIMD.h
to a CMake configured file (see line 42 inCMakeLists.txt
).So it might be a slight inconvenience if someone doesn't want to use CMake.
(I.e. perhaps you don't want to merge this.)