Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

Outdated LLVM is used #6

Open
Jajcus opened this issue Feb 18, 2016 · 6 comments
Open

Outdated LLVM is used #6

Jajcus opened this issue Feb 18, 2016 · 6 comments

Comments

@Jajcus
Copy link

Jajcus commented Feb 18, 2016

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.

@greg-lunarg
Copy link
Contributor

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?

@johnkslang
Copy link
Contributor

There is a 3.6 branch present, which we should figure out soon how to integrate into master.

@Jajcus
Copy link
Author

Jajcus commented Feb 19, 2016

FYI, you do not need to build LunarGLASS for LoaderAndValidationLayers or VulkanSamplesKit. You only need it to build VulkanTools/icd.

Good to know, thanks!

It seems skim-reading of the build docs was not enough.

Or did you want to build VulkanTools/icd?

No, the loader is what I need the most now.

There is a 3.6 branch present, which we should figure out soon how to integrate into master.

Will it work with a vanila upstream LLVM, or will LLVM still be patched for LunarGLASS?

@greg-lunarg
Copy link
Contributor

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.

@gzorin
Copy link

gzorin commented Mar 31, 2016

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.

@Metalhead33
Copy link

Why not forget about LLVM 3.x altogether and support 6 or 7?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants