This is an archive of the discontinued LLVM Phabricator instance.

Differential D122800

[libcxx] [ci] Enable -Werror for libcxxabi and libunwind too
ClosedPublic

Authored by mstorsjo on Mar 31 2022, 3:54 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rG64d7f778e6d7: [libcxx] [ci] Enable -Werror for libcxxabi and libunwind too

Diff Detail

Event Timeline

mstorsjo created this revision.Mar 31 2022, 3:54 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 31 2022, 3:54 AM
Herald added a subscriber: arichardson. · View Herald Transcript
mstorsjo requested review of this revision.Mar 31 2022, 3:54 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 31 2022, 3:54 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Harbormaster completed remote builds in B157139: Diff 419392.Mar 31 2022, 4:56 AM
ldionne accepted this revision.Mar 31 2022, 5:41 AM
This revision is now accepted and ready to land.Mar 31 2022, 5:41 AM
Closed by commit rG64d7f778e6d7: [libcxx] [ci] Enable -Werror for libcxxabi and libunwind too (authored by mstorsjo). · Explain WhyApr 1 2022, 12:15 AM
This revision was automatically updated to reflect the committed changes.
mstorsjo added a commit: rG64d7f778e6d7: [libcxx] [ci] Enable -Werror for libcxxabi and libunwind too.

Revision Contents

PathSize
libcxx/
utils/
ci/
10 lines

Diff 419647

libcxx/utils/ci/run-buildbot

Show First 20 Lines • Show All 78 Lines • ▼ Show 20 Linesfunction generate-cmake-base() {
echo "--- Generating CMake" echo "--- Generating CMake"
${CMAKE} \ ${CMAKE} \
-S "${MONOREPO_ROOT}/runtimes" \ -S "${MONOREPO_ROOT}/runtimes" \
-B "${BUILD_DIR}" \ -B "${BUILD_DIR}" \
-GNinja -DCMAKE_MAKE_PROGRAM="${NINJA}" \ -GNinja -DCMAKE_MAKE_PROGRAM="${NINJA}" \
-DCMAKE_BUILD_TYPE=RelWithDebInfo \ -DCMAKE_BUILD_TYPE=RelWithDebInfo \
-DCMAKE_INSTALL_PREFIX="${INSTALL_DIR}" \ -DCMAKE_INSTALL_PREFIX="${INSTALL_DIR}" \
-DLIBCXX_ENABLE_WERROR=YES \ -DLIBCXX_ENABLE_WERROR=YES \
-DLIBCXXABI_ENABLE_WERROR=YES \
-DLIBUNWIND_ENABLE_WERROR=YES \
-DLLVM_LIT_ARGS="-sv --show-unsupported --xunit-xml-output test-results.xml" \ -DLLVM_LIT_ARGS="-sv --show-unsupported --xunit-xml-output test-results.xml" \
"${@}" "${@}"
} }
function generate-cmake() { function generate-cmake() {
generate-cmake-base \ generate-cmake-base \
-DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind" \ -DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind" \
-DLIBCXX_CXX_ABI=libcxxabi \ -DLIBCXX_CXX_ABI=libcxxabi \
▲ Show 20 Lines • Show All 186 Lines • ▼ Show 20 Linesgeneric-clang-14)
check-runtimes check-runtimes
check-abi-list check-abi-list
;; ;;
generic-gcc) generic-gcc)
export CC=gcc-11 export CC=gcc-11
export CXX=g++-11 export CXX=g++-11
clean clean
generate-cmake -DLIBUNWIND_TEST_CONFIG="llvm-libunwind-shared.cfg.in" \ generate-cmake -DLIBUNWIND_TEST_CONFIG="llvm-libunwind-shared.cfg.in" \
-DLIBCXX_ENABLE_WERROR=NO -DLIBCXX_ENABLE_WERROR=NO \
-DLIBCXXABI_ENABLE_WERROR=NO \
-DLIBUNWIND_ENABLE_WERROR=NO
check-runtimes check-runtimes
;; ;;
generic-gcc-cxx11) generic-gcc-cxx11)
export CC=gcc-11 export CC=gcc-11
export CXX=g++-11 export CXX=g++-11
clean clean
generate-cmake -C "${MONOREPO_ROOT}/libcxx/cmake/caches/Generic-cxx11.cmake" \ generate-cmake -C "${MONOREPO_ROOT}/libcxx/cmake/caches/Generic-cxx11.cmake" \
-DLIBUNWIND_TEST_CONFIG="llvm-libunwind-shared.cfg.in" \ -DLIBUNWIND_TEST_CONFIG="llvm-libunwind-shared.cfg.in" \
-DLIBCXX_ENABLE_WERROR=NO -DLIBCXX_ENABLE_WERROR=NO \
-DLIBCXXABI_ENABLE_WERROR=NO \
-DLIBUNWIND_ENABLE_WERROR=NO
check-runtimes check-runtimes
;; ;;
generic-asan) generic-asan)
clean clean
generate-cmake -C "${MONOREPO_ROOT}/libcxx/cmake/caches/Generic-asan.cmake" \ generate-cmake -C "${MONOREPO_ROOT}/libcxx/cmake/caches/Generic-asan.cmake" \
-DLIBUNWIND_TEST_CONFIG="llvm-libunwind-shared.cfg.in" -DLIBUNWIND_TEST_CONFIG="llvm-libunwind-shared.cfg.in"
check-runtimes check-runtimes
;; ;;
▲ Show 20 Lines • Show All 294 Lines • Show Last 20 Lines