SeExpr is an embeddable, arithmetic expression language that enables flexible artistic control and customization in creating computer graphics images. Example uses include procedural geometry synthesis, image synthesis, simulation control, crowd animation, and geometry deformation. https://wdas.github.io/SeExpr
Another pull request for Windows. This time it handles build SeExpr with LLVM support enabled.
I made a fix on CMakeLists.txt related to LLVM include/lib dirs and version : on my first LLVM builds, for a reason that I ignore, llvm-config wasn't built, even though LLVM was building fine. And I also noticed that in the LLVMConfig.cmake file, variables are already provided with the correct values for include dir, lib dir and version. So I decided to use them instead of calling llvm-config, which might not exist. I left the non-win32 platforms unmodified, although I'm wondering why using llvm-config instead of directly the provided variables ?
There is a serious conflict between LLVM and Windows.h located in the COFF.h file of LLVM. See this Google Groups discussion for more informations: https://groups.google.com/forum/#!topic/llvm-dev/6n5Q0pFdaSA Basically you can't include windows.h in a compilation unit where you also include some LLVM headers ... So I had to remove the #include <Windows.h> statement from Platform.h, and create a proper Platform.cpp file where I could create the Windows implementation without conflicting with LLVM headers. Again, other platforms were left unmodified.
And as always, feel free to comment the PR, ask question or whatever :)
Another pull request for Windows. This time it handles build SeExpr with LLVM support enabled.
I made a fix on CMakeLists.txt related to LLVM include/lib dirs and version : on my first LLVM builds, for a reason that I ignore, llvm-config wasn't built, even though LLVM was building fine. And I also noticed that in the LLVMConfig.cmake file, variables are already provided with the correct values for include dir, lib dir and version. So I decided to use them instead of calling llvm-config, which might not exist. I left the non-win32 platforms unmodified, although I'm wondering why using llvm-config instead of directly the provided variables ?
There is a serious conflict between LLVM and Windows.h located in the COFF.h file of LLVM. See this Google Groups discussion for more informations: https://groups.google.com/forum/#!topic/llvm-dev/6n5Q0pFdaSA Basically you can't include windows.h in a compilation unit where you also include some LLVM headers ... So I had to remove the
#include <Windows.h>
statement fromPlatform.h
, and create a properPlatform.cpp
file where I could create the Windows implementation without conflicting with LLVM headers. Again, other platforms were left unmodified.And as always, feel free to comment the PR, ask question or whatever :)