Skip to content

Commit 176db3b

Browse files
committed
[RFC] Remove support for building C++ with LLVM_ENABLE_PROJECTS
This has been officially deprecated since D112724, meaning the deprecation warning is present in released 14 and 15. This makes me think that now, shortly after the 15 release is branched, is a good time to pull the trigger. Reviewed By: phosek Differential Revision: https://reviews.llvm.org/D132324
1 parent 3917306 commit 176db3b

File tree

6 files changed

+7
-42
lines changed

6 files changed

+7
-42
lines changed

clang/lib/Driver/ToolChains/Linux.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -307,13 +307,6 @@ Linux::Linux(const Driver &D, const llvm::Triple &Triple, const ArgList &Args)
307307

308308
Generic_GCC::AddMultiarchPaths(D, SysRoot, OSLibDir, Paths);
309309

310-
// The deprecated -DLLVM_ENABLE_PROJECTS=libcxx configuration installs
311-
// libc++.so in D.Dir+"/../lib/". Detect this path.
312-
// TODO Remove once LLVM_ENABLE_PROJECTS=libcxx is unsupported.
313-
if (StringRef(D.Dir).startswith(SysRoot) &&
314-
D.getVFS().exists(D.Dir + "/../lib/libc++.so"))
315-
addPathIfExists(D, D.Dir + "/../lib", Paths);
316-
317310
addPathIfExists(D, concat(SysRoot, "/lib"), Paths);
318311
addPathIfExists(D, concat(SysRoot, "/usr/lib"), Paths);
319312
}

compiler-rt/lib/sanitizer_common/symbolizer/scripts/build_symbolizer.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,12 +100,12 @@ if [[ ! -d ${LIBCXX_BUILD} ]]; then
100100
mkdir -p ${LIBCXX_BUILD}
101101
cd ${LIBCXX_BUILD}
102102
LIBCXX_FLAGS="${FLAGS} -Wno-macro-redefined"
103-
PROJECTS=
103+
RUNTIMES=
104104
if [[ ! -d $LLVM_SRC/projects/libcxxabi ]] ; then
105-
PROJECTS="-DLLVM_ENABLE_PROJECTS='libcxx;libcxxabi'"
105+
RUNTIMES="-DLLVM_ENABLE_RUNTIMES='libcxx;libcxxabi'"
106106
fi
107107
cmake -GNinja \
108-
${PROJECTS} \
108+
${RUNTIMES} \
109109
-DCMAKE_BUILD_TYPE=Release \
110110
-DCMAKE_C_COMPILER=$CC \
111111
-DCMAKE_CXX_COMPILER=$CXX \

libcxx/utils/ci/buildkite-pipeline.yml

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -321,20 +321,6 @@ steps:
321321
limit: 2
322322
timeout_in_minutes: 120
323323

324-
- label: "Legacy LLVM_ENABLE_PROJECTS build"
325-
command: "libcxx/utils/ci/run-buildbot legacy-project-build"
326-
artifact_paths:
327-
- "**/test-results.xml"
328-
- "**/*.abilist"
329-
agents:
330-
queue: "libcxx-builders"
331-
os: "linux"
332-
retry:
333-
automatic:
334-
- exit_status: -1 # Agent was lost
335-
limit: 2
336-
timeout_in_minutes: 120
337-
338324
# Tests with various build configurations.
339325
- label: "Static libraries"
340326
command: "libcxx/utils/ci/run-buildbot generic-static"

libcxx/utils/ci/oss-fuzz.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ INSTALL=cxx_install_dir
1313

1414
mkdir ${BUILD}
1515
cmake -S ${PWD} -B ${BUILD} \
16-
-DLLVM_ENABLE_PROJECTS="libcxx;libcxxabi" \
16+
-DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi" \
1717
-DCMAKE_BUILD_TYPE=RelWithDebInfo \
1818
-DCMAKE_INSTALL_PREFIX="${INSTALL}"
1919
cmake --build ${BUILD} --target install-cxx-headers

libcxx/utils/ci/run-buildbot

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -510,21 +510,6 @@ legacy-test-config)
510510
-DLIBUNWIND_TEST_CONFIG="${MONOREPO_ROOT}/libunwind/test/lit.site.cfg.in"
511511
check-runtimes
512512
;;
513-
legacy-project-build)
514-
clean
515-
516-
echo "--- Generating CMake"
517-
${CMAKE} \
518-
-S "${MONOREPO_ROOT}/llvm" \
519-
-B "${BUILD_DIR}" \
520-
-DLLVM_ENABLE_PROJECTS="libcxx;libunwind;libcxxabi" \
521-
-GNinja -DCMAKE_MAKE_PROGRAM="${NINJA}" \
522-
-DCMAKE_BUILD_TYPE=RelWithDebInfo \
523-
-DCMAKE_INSTALL_PREFIX="${INSTALL_DIR}" \
524-
-DLLVM_LIT_ARGS="-sv --show-unsupported --xunit-xml-output test-results.xml --timeout=1500" \
525-
-DLIBCXX_CXX_ABI=libcxxabi
526-
check-runtimes
527-
;;
528513
aarch64)
529514
clean
530515
generate-cmake -C "${MONOREPO_ROOT}/libcxx/cmake/caches/AArch64.cmake"

llvm/CMakeLists.txt

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,9 @@ foreach(proj ${LLVM_ENABLE_PROJECTS})
123123
endforeach()
124124
foreach(proj "libcxx" "libcxxabi" "libunwind")
125125
if (${proj} IN_LIST LLVM_ENABLE_PROJECTS)
126-
message(WARNING "Using LLVM_ENABLE_PROJECTS=${proj} is deprecated now, please use -DLLVM_ENABLE_RUNTIMES=${proj} or "
127-
"see the instructions at https://libcxx.llvm.org/BuildingLibcxx.html for building the runtimes.")
126+
message(FATAL_ERROR
127+
"Using LLVM_ENABLE_PROJECTS=${proj} is incorrect. Please use -DLLVM_ENABLE_RUNTIMES=${proj} or "
128+
"see the instructions at https://libcxx.llvm.org/BuildingLibcxx.html for building the runtimes.")
128129
endif()
129130
endforeach()
130131

0 commit comments

Comments
 (0)