Skip to content

Commit

Permalink
remove LLVM < 6 from build/test scripts
Browse files Browse the repository at this point in the history
  • Loading branch information
251 authored and ccadar committed Mar 17, 2022
1 parent d8553f3 commit 0f45ab7
Show file tree
Hide file tree
Showing 18 changed files with 14 additions and 440 deletions.
17 changes: 0 additions & 17 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,6 @@ jobs:
"LLVM 8",
"LLVM 7",
"LLVM 6",
"LLVM 5",
"LLVM 4",
"LLVM 3.9",
"LLVM 3.8",
"ASan",
"UBSan",
"MSan",
Expand Down Expand Up @@ -84,19 +80,6 @@ jobs:
- name: "LLVM 6"
env:
LLVM_VERSION: 6
- name: "LLVM 5"
env:
LLVM_VERSION: 5
- name: "LLVM 4"
env:
LLVM_VERSION: 4
- name: "LLVM 3.9"
env:
LLVM_VERSION: 3.9
- name: "LLVM 3.8"
env:
LLVM_VERSION: 3.8
USE_LIBCXX: 0
# Sanitizer builds. Do unoptimized build otherwise the optimizer might remove problematic code
- name: "ASan"
env:
Expand Down
3 changes: 1 addition & 2 deletions cmake/workaround_llvm_pr39177.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,7 @@
# [MODIFIED] lib/Module/Passes.h

# Detect whether LLVM version is affected by PR39177
if ((${LLVM_VERSION_MAJOR} GREATER 3 OR (${LLVM_VERSION_MAJOR} EQUAL 3 AND ${LLVM_VERSION_MINOR} EQUAL 9)) # LLVM >= 3.9
AND (${LLVM_VERSION_MAJOR} LESS 7 OR (${LLVM_VERSION_MAJOR} EQUAL 7 AND ${LLVM_VERSION_MINOR} EQUAL 0 AND ${LLVM_VERSION_PATCH} EQUAL 0))) # LLVM <= 7.0.0
if (${LLVM_VERSION_MAJOR} LESS 7 OR (${LLVM_VERSION_MAJOR} EQUAL 7 AND ${LLVM_VERSION_MINOR} EQUAL 0 AND ${LLVM_VERSION_PATCH} EQUAL 0)) # LLVM <= 7.0.0
set(DISABLE_WORKAROUND_LLVM_PR39177_DEFAULT OFF)
else()
set(DISABLE_WORKAROUND_LLVM_PR39177_DEFAULT ON)
Expand Down
6 changes: 2 additions & 4 deletions runtime/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -79,12 +79,10 @@ set(COMMON_CC_FLAGS
-D__STDC_CONSTANT_MACROS
-Wall
-Wwrite-strings
-Xclang
-disable-O0-optnone
)

if (${LLVM_VERSION_MAJOR} GREATER 4)
list(APPEND COMMON_CC_FLAGS "-Xclang" "-disable-O0-optnone")
endif ()

foreach (_suffix ${LIB_BC_SUFFIX})
list(APPEND "LIB_BC_FLAGS_${_suffix}" ${COMMON_CC_FLAGS})
endforeach ()
Expand Down
8 changes: 2 additions & 6 deletions scripts/build/p-libcxx.inc
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,7 @@ download_libcxx() {
[[ -f "${LIBCXX_SRC}/.src_checked_out" ]] && return 0

local LLVM_VERSION_MAJOR="${LLVM_VERSION/.*/}"

local version=""
[[ "${LLVM_VERSION_MAJOR}" -le 3 ]] && version="release/${LLVM_VERSION}.x"
[[ "${LLVM_VERSION_MAJOR}" -ge 4 ]] && version="release/${LLVM_VERSION_MAJOR}.x"

local version="release/${LLVM_VERSION_MAJOR}.x"

git clone --single-branch --branch "${version}" --depth 1 "https://github.com/llvm/llvm-project.git" "${LIBCXX_SRC}"

Expand Down Expand Up @@ -85,7 +81,7 @@ install_libcxx() {
fi

local LLVM_VERSION_MAJOR="${LLVM_VERSION/.*/}"
[[ "${LLVM_VERSION_MAJOR}" -ge 6 ]] && libraries+=("${LIBCXX_INSTALL}"/lib/lib*.a)
libraries+=("${LIBCXX_INSTALL}"/lib/lib*.a)


for p in "${libraries[@]}" ; do
Expand Down
25 changes: 3 additions & 22 deletions scripts/build/p-llvm.inc
Original file line number Diff line number Diff line change
Expand Up @@ -97,17 +97,7 @@ download_llvm() {
local LLVM_VERSION_MAJOR="${LLVM_VERSION/.*/}"

branch_name="release/${LLVM_VERSION_MAJOR}.x"
if [[ "${LLVM_VERSION_MAJOR}" -le 3 ]]; then
branch_name="release/${LLVM_VERSION}.x"
fi
git_clone_or_update "https://github.com/llvm/llvm-project.git" "${LLVM_SRC_BASE}" "${branch_name}" || exit 1
if [[ "${LLVM_VERSION_MAJOR}" -lt 4 ]]; then
# Use symlinks for older versions whose build systems do not support the monorepo directory layout
ln -s "${LLVM_SRC_BASE}/clang" "${LLVM_SRC_BASE}/llvm/tools/"
ln -s "${LLVM_SRC_BASE}/compiler-rt" "${LLVM_SRC_BASE}/llvm/projects/"
ln -s "${LLVM_SRC_BASE}/libcxx" "${LLVM_SRC_BASE}/llvm/projects/"
ln -s "${LLVM_SRC_BASE}/libcxxabi" "${LLVM_SRC_BASE}/llvm/projects/"
fi

# Apply existing patches if needed
if [ -f "${DIR}/patches/llvm${LLVM_VERSION_SHORT}.patch" ]; then
Expand Down Expand Up @@ -190,11 +180,7 @@ build_llvm() {
)

# Select components to build
if [[ ${LLVM_VERSION_SHORT} -ge 60 ]]; then
CONFIG+=("-DLLVM_ENABLE_PROJECTS=clang;compiler-rt;libcxx;libcxxabi")
else
CONFIG+=("-DLLVM_ENABLE_PROJECTS=clang;compiler-rt")
fi
CONFIG+=("-DLLVM_ENABLE_PROJECTS=clang;compiler-rt;libcxx;libcxxabi")

# cmake build
if [[ "${enable_optimized}" == "1" && "${enable_debug}" != "1" ]]; then
Expand Down Expand Up @@ -279,13 +265,8 @@ install_llvm() {
install-clang install-llvm-config install-llvm-objdump
install-llvm-link install-llvm-ar install-llvm-nm install-llvm-dis
install-clang-headers install-llvm-as
install-llvm-symbolizer install-LLVMSupport install-lli not FileCheck )

if [[ ${LLVM_VERSION_SHORT} -eq 38 ]]; then
LLVM_PACKAGES=("${LLVM_PACKAGES[@]}" installhdrs)
else
LLVM_PACKAGES=("${LLVM_PACKAGES[@]}" install-llvm-headers)
fi
install-llvm-symbolizer install-LLVMSupport install-lli not FileCheck
install-llvm-headers )

ninja "${LLVM_PACKAGES[@]}"

Expand Down
11 changes: 0 additions & 11 deletions scripts/build/patches/libcxx38.patch

This file was deleted.

11 changes: 0 additions & 11 deletions scripts/build/patches/libcxx39.patch

This file was deleted.

11 changes: 0 additions & 11 deletions scripts/build/patches/libcxx40.patch

This file was deleted.

127 changes: 0 additions & 127 deletions scripts/build/patches/llvm38.patch

This file was deleted.

76 changes: 0 additions & 76 deletions scripts/build/patches/llvm39.patch

This file was deleted.

Loading

0 comments on commit 0f45ab7

Please sign in to comment.