[libcxx-commits] [libcxx] [libc++] Add a CI job for the LLDB data formatters (PR #65174)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Oct 12 16:50:27 PDT 2023
https://github.com/ldionne updated https://github.com/llvm/llvm-project/pull/65174
>From 9ff29c54e97dab71488ffe50ac871f007d357e4f Mon Sep 17 00:00:00 2001
From: Louis Dionne <ldionne.2 at gmail.com>
Date: Fri, 9 Sep 2022 19:12:18 -0400
Subject: [PATCH 1/2] [libc++] Add a CI job for the LLDB data formatters
This patch adds a CI job running the LLDB data formatters in the libc++
CI. This should avoid breaking the LLDB data formatters when e.g. the
layout of libc++ data structures is changed.
This was originally opened as https://reviews.llvm.org/D133621.
---
libcxx/utils/ci/buildkite-pipeline.yml | 16 ++++++++++++++++
libcxx/utils/ci/run-buildbot | 14 ++++++++++++++
2 files changed, 30 insertions(+)
diff --git a/libcxx/utils/ci/buildkite-pipeline.yml b/libcxx/utils/ci/buildkite-pipeline.yml
index 723ae438619745f..929838cd91ae594 100644
--- a/libcxx/utils/ci/buildkite-pipeline.yml
+++ b/libcxx/utils/ci/buildkite-pipeline.yml
@@ -728,6 +728,22 @@ steps:
limit: 2
timeout_in_minutes: 120
+ - label: "LLDB Data Formatters"
+ command: "libcxx/utils/ci/run-buildbot check-lldb-data-formatters"
+ artifact_paths:
+ - "**/test-results.xml"
+ env:
+ CC: "clang-${LLVM_HEAD_VERSION}"
+ CXX: "clang++-${LLVM_HEAD_VERSION}"
+ agents:
+ queue: "libcxx-builders"
+ os: "linux"
+ retry:
+ automatic:
+ - exit_status: -1 # Agent was lost
+ limit: 2
+ timeout_in_minutes: 120
+
# Other non-testing CI jobs
- label: "Benchmarks"
command: "libcxx/utils/ci/run-buildbot benchmarks"
diff --git a/libcxx/utils/ci/run-buildbot b/libcxx/utils/ci/run-buildbot
index fb4c5e1bfcf65be..f7edf436cf6c391 100755
--- a/libcxx/utils/ci/run-buildbot
+++ b/libcxx/utils/ci/run-buildbot
@@ -574,6 +574,20 @@ benchmarks)
generate-cmake
check-cxx-benchmarks
;;
+check-lldb-data-formatters)
+ clean
+ ${CMAKE} \
+ -S "${MONOREPO_ROOT}/llvm" \
+ -B "${BUILD_DIR}" \
+ -GNinja -DCMAKE_MAKE_PROGRAM="${NINJA}" \
+ -DCMAKE_BUILD_TYPE=RelWithDebInfo \
+ -DLLVM_ENABLE_ASSERTIONS=ON \
+ -DLLDB_ENABLE_PYTHON=ON \
+ -DLLVM_ENABLE_PROJECTS='llvm;clang;lldb' \
+ -DLLVM_ENABLE_RUNTIMES='libcxx;libcxxabi'
+
+ ${NINJA} -C "${BUILD_DIR}" check-lldb-api-functionalities-data-formatter-data-formatter-stl-libcxx
+;;
documentation)
clean
generate-cmake -DLLVM_ENABLE_SPHINX=ON
>From 63879abeb3d62494b5602c66afcc647674acbe78 Mon Sep 17 00:00:00 2001
From: Louis Dionne <ldionne.2 at gmail.com>
Date: Thu, 12 Oct 2023 16:50:06 -0700
Subject: [PATCH 2/2] [libc++] Upload artifacts of the Bootstrapping build to
BK
---
libcxx/utils/ci/run-buildbot | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/libcxx/utils/ci/run-buildbot b/libcxx/utils/ci/run-buildbot
index dd387357f0cd86b..705f6969f44142d 100755
--- a/libcxx/utils/ci/run-buildbot
+++ b/libcxx/utils/ci/run-buildbot
@@ -367,6 +367,10 @@ bootstrapping-build)
-DLLVM_ENABLE_ASSERTIONS=ON \
-DLLVM_LIT_ARGS="-v --show-unsupported --xunit-xml-output test-results.xml --timeout=1500 --time-tests"
+ ${NINJA} -vC "${BUILD_DIR}" install-clang install-clang-resource-headers
+ tar -cJvf install.tar.xz ${INSTALL_DIR}
+ buildkite-agent artifact upload --debug install.tar.xz
+
echo "+++ Running the libc++ and libc++abi tests"
${NINJA} -vC "${BUILD_DIR}" check-runtimes
More information about the libcxx-commits
mailing list