add build recipe for LLVM9

[vchuravy]

Based on https://github.com/staticfloat/LLVMBuilder

Questions:

• Do we want a recipe for LLVM6? It's part of LTS so maybe
• This removes the assert build configuration

[giordano]

Should we update the pipeline to skip this? The @ wasn't initially expected.

[vchuravy]

@staticfloat would an assert build be something like LLVM@8.asserts? Is there a way for them to share the same script?

[vchuravy]

Okay reverted back to splitting the native and cross-compile stage:

For future reference I almost got Native to work on windows, but got stuck on include errors.

vchuravy@antarctic:/data/vchuravy/Yggdrasil/L/LLVM@8$ cat bundled/patches/7003-llvm-cmake-mingw.patch
---
 llvm/cmake/modules/AddLLVM.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cmake/modules/AddLLVM.cmake b/cmake/modules/AddLLVM.cmake
index 1a417447278..57ab2576574 100644
--- a/cmake/modules/AddLLVM.cmake
+++ b/cmake/modules/AddLLVM.cmake
@@ -648,7 +648,7 @@ macro(add_llvm_library name)
         (LLVM_LINK_LLVM_DYLIB AND ${name} STREQUAL "LLVM"))
       set(install_dir lib${LLVM_LIBDIR_SUFFIX})
       if(ARG_MODULE OR ARG_SHARED OR BUILD_SHARED_LIBS)
-        if(WIN32 OR CYGWIN OR MINGW)
+        if((WIN32 OR CYGWIN OR MINGW) AND NOT LLVM_TARGET_IS_CROSSCOMPILE_HOST)
           set(install_type RUNTIME)
           set(install_dir bin)
         else()
--
2.24.1

[vchuravy]

Ah is Yggdrasil doing some sneaky parsing and having multiple platforms statements is confusing?

[tshort]

Any chance we could include the WebAssembly target with this?

[staticfloat]

I am so confused by this error.

[vchuravy]

Any chance we could include the WebAssembly target with this?

Actually yes, that is needed since Base Julia know has this flag on by default.

[Gnimuc]

Any chance we could include the WebAssembly target with this?

Actually yes, that is needed since Base Julia now has this flag on by default.

Any plan on maintaining a Julia-release-compatible LLVM recipe? Julia 1.3.0 is built with WASM disabled.

[vchuravy]

Any chance we could include the WebAssembly target with this?

Actually yes, that is needed since Base Julia now has this flag on by default.

Any plan on maintaining a Julia-release-compatible LLVM recipe? Julia 1.3.0 is built with WASM disabled.

Actually yes! We only build WASM for LLVM 8 which is Julia 1.4 only. See https://github.com/JuliaPackaging/Yggdrasil/pull/296/files#diff-08724ed72679c754126058e3bfd19c92R111-R113

[staticfloat]

Valentin, just so you know, I needed to multiversion OpenBLAS as well, and I decided that I liked putting the common stuff into an OpenBLAS directory, with OpenBLAS@0.3.5 and OpenBLAS@0.3.7 doing include("../OpenBLAS/common.jl"), see #356

[vchuravy]

I think there are two patches that still need forward porting got LLVM 9, but otherwise I am rather pleased to have builds for all platforms. LLVM8 and co. can follow later

[staticfloat]

This is pretty incredible. Shall we merge this, then we can open separate PRs for v6 and v8?

[giordano]

Great job 🙂

[giordano]

Update the title? Or at least the commit message when merging

[vchuravy]

Ready to land from my side.

[staticfloat]

[aminya]

Yay! 🚀