Skip to content

Commit

Permalink
Fix llvm-tools when installing with LLVM_ASSERTS=1 (JuliaLang#40447)
Browse files Browse the repository at this point in the history
When installing with `LLVM_ASSERTS=1`, we need our `llvm-tools` to be an
assert build as well.
  • Loading branch information
staticfloat authored Apr 22, 2021
1 parent 2c88fb7 commit 15a44e9
Show file tree
Hide file tree
Showing 4 changed files with 180 additions and 117 deletions.
4 changes: 3 additions & 1 deletion contrib/refresh_checksums.mk
Original file line number Diff line number Diff line change
Expand Up @@ -73,9 +73,11 @@ $(foreach project,$(BB_GCC_EXPANDED_PROJECTS),$(foreach triplet,$(TRIPLETS),$(fo
$(foreach project,$(BB_CXX_EXPANDED_PROJECTS),$(foreach triplet,$(NON_CLANG_TRIPLETS),$(foreach cxxstring_abi,cxx11 cxx03,$(eval $(call checksum_dep,$(project),$(triplet)-$(cxxstring_abi))))))
$(foreach project,$(BB_CXX_EXPANDED_PROJECTS),$(foreach triplet,$(CLANG_TRIPLETS),$(eval $(call checksum_dep,$(project),$(triplet)))))

# Special libLLVM_asserts_jll targets
# Special libLLVM_asserts_jll/LLVM_assert_jll targets
$(foreach triplet,$(NON_CLANG_TRIPLETS),$(foreach cxxstring_abi,cxx11 cxx03,$(eval $(call checksum_dep,llvm,$(triplet)-$(cxxstring_abi),assert))))
$(foreach triplet,$(NON_CLANG_TRIPLETS),$(foreach cxxstring_abi,cxx11 cxx03,$(eval $(call checksum_dep,llvm-tools,$(triplet)-$(cxxstring_abi),assert))))
$(foreach triplet,$(CLANG_TRIPLETS),$(eval $(call checksum_dep,llvm,$(triplet),assert)))
$(foreach triplet,$(CLANG_TRIPLETS),$(eval $(call checksum_dep,llvm-tools,$(triplet),assert)))

# External stdlibs
checksum-stdlibs:
Expand Down
1 change: 1 addition & 0 deletions deps/Versions.make
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ LLVM_JLL_NAME := libLLVM
# LLVM_tools (downloads LLVM_jll to get things like `lit` and `opt`)
LLVM_TOOLS_JLL_NAME := LLVM
LLVM_TOOLS_JLL_VER := 11.0.1+3
LLVM_TOOLS_ASSERT_JLL_VER := 11.0.1+3

# LLVM libunwind
LLVMUNWIND_VER := 11.0.1
Expand Down
Loading

0 comments on commit 15a44e9

Please sign in to comment.