-
Notifications
You must be signed in to change notification settings - Fork 22
Outdated LLVM is used #6
Comments
FYI, you do not need to build LunarGLASS for LoaderAndValidationLayers or VulkanSamplesKit. You only need it to build VulkanTools/icd. If you are not interested in building VulkanTools/icd, we are in the process of solving this problem by changing update_external_sources.sh in LoaderAndValidationLayers so that it no longer builds LunarGLASS. In the meantime, there are options to update_external_sources.sh to avoid building LunarGLASS. Or did you want to build VulkanTools/icd? |
There is a 3.6 branch present, which we should figure out soon how to integrate into master. |
Good to know, thanks! It seems skim-reading of the build docs was not enough.
No, the loader is what I need the most now.
Will it work with a vanila upstream LLVM, or will LLVM still be patched for LunarGLASS? |
LunarGLASS is no longer built for LoaderAndValidationLayers on the sdk-1.0.3 branch. It has been moved to update_external_sources.sh in VulkanTools. I will keep this open to track the issue that a new version of LLVM in LunarGLASS would be desirable, although it should be noted that VulkanTools builds and statically links to LLVM from source, which is available at version 3.4 through the LLVM web site. |
I've been working on a branch that uses LLVM 3.8: https://github.com/gzorin/LunarGLASS/tree/LLVM3.8 Right now the tests run without crashing, and running LunarGOO individually on many of the test sources successfully produces GLSL. However 'runtests' reports that most (all?) of the test suite fails, and I haven't sorted out how much of that's actual failure introduced by my changes & how much is due to differences in the LLVM IR or GLSL that are otherwise correct. |
Why not forget about LLVM 3.x altogether and support 6 or 7? |
LunarGLASS build requires LLVM 3.4, while current version is 3.7.1 and some Linux distribution don't ship the old 3.4 any more. This makes LunarGLASS unreasonably hard to package for current Linux distributions, which is painful, as this project is one of the dependencies of the Vulkan SDK.
The text was updated successfully, but these errors were encountered: