Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Force builds against libLLVM.dylib and drop LLVM 7 #936

Merged
merged 1 commit into from
Jan 6, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/ci-ubuntu-18.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
container: ubuntu:18.04
strategy:
matrix:
LLVM_VERSION: ["7.0", "8.0", "9.0", "10.0"]
LLVM_VERSION: ["8.0", "9.0", "10.0"]

steps:
- name: For debugging
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
container: ubuntu:20.04
strategy:
matrix:
LLVM_VERSION: ["7.0", "8.0", "9.0", "10.0", "11.0", "12.0", "13.0"]
LLVM_VERSION: ["8.0", "9.0", "10.0", "11.0", "12.0"]

steps:
- name: Install software
Expand Down
11 changes: 0 additions & 11 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -202,17 +202,6 @@ set (MULL_INCLUDE_DIRS
${CMAKE_CURRENT_LIST_DIR}/LLVMCompatibility/${LLVM_COMPATIBILITY_DIR}
)

if ("LLVM" IN_LIST LLVM_AVAILABLE_LIBS)
set (MULL_LLVM_COMPATIBILITY_LIBRARIES
LLVM
)
else()
set (MULL_LLVM_COMPATIBILITY_LIBRARIES
LLVMSupport
LLVMCore
)
endif()

include(${CMAKE_CURRENT_LIST_DIR}/cmake/cpack.cmake)
include(${CMAKE_CURRENT_LIST_DIR}/cmake/version.cmake)

Expand Down
1 change: 0 additions & 1 deletion LLVMCompatibility/10.x.x/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ set_target_properties(LLVMCompatibility PROPERTIES
target_include_directories(LLVMCompatibility SYSTEM PUBLIC
${LLVM_INCLUDE_DIRS}
)
add_dependencies(LLVMCompatibility ${MULL_LLVM_COMPATIBILITY_LIBRARIES})
1 change: 0 additions & 1 deletion LLVMCompatibility/11.x.x/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ set_target_properties(LLVMCompatibility PROPERTIES
target_include_directories(LLVMCompatibility SYSTEM PUBLIC
${LLVM_INCLUDE_DIRS}
)
add_dependencies(LLVMCompatibility ${MULL_LLVM_COMPATIBILITY_LIBRARIES})
1 change: 0 additions & 1 deletion LLVMCompatibility/12.x.x/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ set_target_properties(LLVMCompatibility PROPERTIES
target_include_directories(LLVMCompatibility SYSTEM PUBLIC
${LLVM_INCLUDE_DIRS}
)
add_dependencies(LLVMCompatibility ${MULL_LLVM_COMPATIBILITY_LIBRARIES})
1 change: 0 additions & 1 deletion LLVMCompatibility/13.x.x/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ set_target_properties(LLVMCompatibility PROPERTIES
target_include_directories(LLVMCompatibility SYSTEM PUBLIC
${LLVM_INCLUDE_DIRS}
)
add_dependencies(LLVMCompatibility ${MULL_LLVM_COMPATIBILITY_LIBRARIES})
13 changes: 0 additions & 13 deletions LLVMCompatibility/7.x.x/CMakeLists.txt

This file was deleted.

69 changes: 0 additions & 69 deletions LLVMCompatibility/7.x.x/LLVMCompatibility.cpp

This file was deleted.

36 changes: 0 additions & 36 deletions LLVMCompatibility/7.x.x/LLVMCompatibility.h

This file was deleted.

1 change: 0 additions & 1 deletion LLVMCompatibility/8.x.x/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ set_target_properties(LLVMCompatibility PROPERTIES
target_include_directories(LLVMCompatibility SYSTEM PUBLIC
${LLVM_INCLUDE_DIRS}
)
add_dependencies(LLVMCompatibility ${MULL_LLVM_COMPATIBILITY_LIBRARIES})
1 change: 0 additions & 1 deletion LLVMCompatibility/9.x.x/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ set_target_properties(LLVMCompatibility PROPERTIES
target_include_directories(LLVMCompatibility SYSTEM PUBLIC
${LLVM_INCLUDE_DIRS}
)
add_dependencies(LLVMCompatibility ${MULL_LLVM_COMPATIBILITY_LIBRARIES})
2 changes: 0 additions & 2 deletions cmake/packaging/cpack.DEB-ubuntu.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,10 @@ set (CPACK_GENERATOR DEB)
set (CPACK_DEBIAN_PACKAGE_MAINTAINER ${CPACK_PACKAGE_CONTACT})
set (CPACK_DEBIAN_PACKAGE_DESCRIPTION ${PROJECT_DESCRIPTION})


if (NOT NO_PRECOMPILED_LLVM)
set (deps "zlib1g, libncurses5, libstdc++6, libxml2, libsqlite3-0")
else()
set (deps "libllvm${LLVM_VERSION_MAJOR}, libxml2, libsqlite3-0")
endif()

set (CPACK_DEBIAN_PACKAGE_DEPENDS ${deps})

26 changes: 9 additions & 17 deletions infrastructure/helpers/variables.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,6 @@ llvm_mapping:
precompiled: true
path: /usr/local/opt/llvm@8
brew_llvm_package: llvm@8
7.0.0:
url: https://releases.llvm.org/7.0.0/clang+llvm-7.0.0-x86_64-apple-darwin.tar.xz
precompiled: true
path: /usr/local/opt/llvm@7
brew_llvm_package: llvm@7

debian:
6.0.0:
Expand All @@ -46,7 +41,15 @@ llvm_mapping:
13.0.0:
url: https://github.com/llvm/llvm-project/releases/download/llvmorg-13.0.0/clang+llvm-13.0.0-x86_64-linux-gnu-ubuntu-20.04.tar.xz
path: clang+llvm-13.0.0-x86_64-linux-gnu-ubuntu-20.04
precompiled: true
precompiled: false
packages:
- llvm-13-dev
- libclang-13-dev
- libc++-13-dev
- libc++abi-13-dev
- clang-13
cmake_cc: clang-13
cmake_cxx: clang++-13
12.0.0:
url: https://github.com/llvm/llvm-project/releases/download/llvmorg-12.0.0/clang+llvm-12.0.0-x86_64-linux-gnu-ubuntu-20.04.tar.xz
path: clang+llvm-12.0.0-x86_64-linux-gnu-ubuntu-20.04
Expand Down Expand Up @@ -107,17 +110,6 @@ llvm_mapping:
- clang-8
cmake_cc: clang-8
cmake_cxx: clang++-8
7.0.0:
url: https://releases.llvm.org/7.0.0/clang+llvm-7.0.0-x86_64-linux-gnu-ubuntu-16.04.tar.xz
path: clang+llvm-7.0.0-x86_64-linux-gnu-ubuntu-16.04
precompiled: true
packages:
- llvm-7-dev
- libclang-7-dev
- libc++-7-dev
- clang-7
cmake_cc: clang-7
cmake_cxx: clang++-7

fedora:
6.0.0:
Expand Down
20 changes: 1 addition & 19 deletions lib/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -120,29 +120,11 @@ add_library(mull

set_source_files_properties(${mull_header_dirs} PROPERTIES HEADER_FILE_ONLY ON)

if ("LLVM" IN_LIST LLVM_AVAILABLE_LIBS)
set(MULL_LLVM_LIBRARIES
LLVM
)
else ()
set(MULL_LLVM_LIBRARIES
LLVMSupport
LLVMOption
LLVMCoverage
LLVMProfileData
LLVMAsmParser
LLVMSupport
LLVMOption
LLVM${LLVM_NATIVE_ARCH}CodeGen
LLVM${LLVM_NATIVE_ARCH}AsmParser
)
endif ()

find_package(Threads REQUIRED)
find_package(SQLite3 REQUIRED)

target_link_libraries(mull
${MULL_LLVM_LIBRARIES}
LLVM
SQLite::SQLite3
clangTooling
Threads::Threads
Expand Down
12 changes: 1 addition & 11 deletions tools/mull-cxx/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,19 +3,9 @@ set (SOURCES
${CMAKE_CURRENT_LIST_DIR}/../CLIOptions/CLIOptions.cpp
)

if ("LLVM" IN_LIST LLVM_AVAILABLE_LIBS)
set (MULL_CXX_LLVM_LIBRARIES
LLVM
)
else()
set (MULL_CXX_LLVM_LIBRARIES
LLVMObject
)
endif()

add_mull_executable(
NAME mull-cxx-${LLVM_VERSION_MAJOR}
SOURCES ${SOURCES}
LINK_WITH ebc mull ${MULL_CXX_LLVM_LIBRARIES} json11
LINK_WITH ebc mull json11 LLVM
)
target_include_directories(mull-cxx-${LLVM_VERSION_MAJOR} PRIVATE ${CMAKE_CURRENT_LIST_DIR}/../CLIOptions)
2 changes: 1 addition & 1 deletion tools/mull-runner/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,6 @@ set (SOURCES
add_mull_executable(
NAME mull-runner-${LLVM_VERSION_MAJOR}
SOURCES ${SOURCES}
LINK_WITH mull json11
LINK_WITH mull json11 LLVM
)
target_include_directories(mull-runner-${LLVM_VERSION_MAJOR} PRIVATE ${CMAKE_CURRENT_LIST_DIR}/../CLIOptions)