https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/94325
>From fe6cde45a5e1f30dc76da872f24ca08d5888e20b Mon Sep 17 00:00:00 2001
From: David Spickett <david.spick...@linaro.org>
Date: Tue, 4 Jun 2024 08:45:41 +0000
Subject: [PATCH 1/4] Enable lldb build
---
.ci/generate-buildkite-pipeline-premerge | 3 +--
.ci/monolithic-linux.sh | 1 +
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/.ci/generate-buildkite-pipeline-premerge
b/.ci/generate-buildkite-pipeline-premerge
index 033ab804b165e..5f8700efabfab 100755
--- a/.ci/generate-buildkite-pipeline-premerge
+++ b/.ci/generate-buildkite-pipeline-premerge
@@ -170,7 +170,6 @@ function exclude-windows() {
compiler-rt) ;; # tests taking too long
openmp) ;; # TODO: having trouble with the Perl installation
libc) ;; # no Windows support
- lldb) ;; # tests failing
bolt) ;; # tests are not supported yet
*)
echo "${project}"
@@ -213,7 +212,7 @@ function check-targets() {
echo "check-unwind"
;;
lldb)
- echo "check-all" # TODO: check-lldb may not include all the LLDB tests?
+ echo "check-lldb"
;;
pstl)
echo "check-all"
diff --git a/.ci/monolithic-linux.sh b/.ci/monolithic-linux.sh
index 38d7128f241b6..b78dc59432b65 100755
--- a/.ci/monolithic-linux.sh
+++ b/.ci/monolithic-linux.sh
@@ -39,6 +39,7 @@ targets="${2}"
echo "--- cmake"
pip install -q -r "${MONOREPO_ROOT}"/mlir/python/requirements.txt
+pip install -q -r "${MONOREPO_ROOT}"/lldb/test/requirements.txt
cmake -S "${MONOREPO_ROOT}"/llvm -B "${BUILD_DIR}" \
-D LLVM_ENABLE_PROJECTS="${projects}" \
-G Ninja \
>From 6f90aea93cf1b2d6b8afad985f5f7168a4dc1da9 Mon Sep 17 00:00:00 2001
From: David Spickett <david.spick...@linaro.org>
Date: Tue, 4 Jun 2024 08:46:45 +0000
Subject: [PATCH 2/4] Touch failing file.
---
.../Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git
a/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
b/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
index 24111396b0ac6..e1ea75bd0317c 100644
--- a/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
+++ b/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
@@ -614,3 +614,4 @@ TEST_F(GDBRemoteCommunicationClientTest, CalculateMD5) {
EXPECT_EQ(expected_low, result->low());
EXPECT_EQ(expected_high, result->high());
}
+
>From be9cb678e7d96c1e34f3df45ae1138fc52fc888d Mon Sep 17 00:00:00 2001
From: David Spickett <david.spick...@linaro.org>
Date: Tue, 4 Jun 2024 09:32:22 +0000
Subject: [PATCH 3/4] Work around msvc problem
---
.../Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git
a/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
b/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
index e1ea75bd0317c..0e45dd5f39257 100644
--- a/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
+++ b/lldb/unittests/Process/gdb-remote/GDBRemoteCommunicationClientTest.cpp
@@ -593,6 +593,10 @@ TEST_F(GDBRemoteCommunicationClientTest, WriteMemoryTags) {
"E03", false);
}
+// Prior to this verison, constructing a std::future for a type without a
+// default constructor is not possible.
+//
https://developercommunity.visualstudio.com/t/c-shared-state-futuresstate-default-constructs-the/60897
+#if _MSC_VER >= 1932
TEST_F(GDBRemoteCommunicationClientTest, CalculateMD5) {
FileSpec file_spec("/foo/bar", FileSpec::Style::posix);
std::future<ErrorOr<MD5::MD5Result>> async_result = std::async(
@@ -614,4 +618,4 @@ TEST_F(GDBRemoteCommunicationClientTest, CalculateMD5) {
EXPECT_EQ(expected_low, result->low());
EXPECT_EQ(expected_high, result->high());
}
-
+#endif
\ No newline at end of file
>From 137ee24f10e9e62b29bf3ea6ac69cef21f2395ae Mon Sep 17 00:00:00 2001
From: David Spickett <david.spick...@linaro.org>
Date: Tue, 4 Jun 2024 09:50:37 +0000
Subject: [PATCH 4/4] Build lld to use in tests and maybe avoid the one in
program files which has a space in the path.
---
.ci/generate-buildkite-pipeline-premerge | 3 +++
1 file changed, 3 insertions(+)
diff --git a/.ci/generate-buildkite-pipeline-premerge
b/.ci/generate-buildkite-pipeline-premerge
index 5f8700efabfab..90cd48359939c 100755
--- a/.ci/generate-buildkite-pipeline-premerge
+++ b/.ci/generate-buildkite-pipeline-premerge
@@ -141,6 +141,9 @@ function add-dependencies() {
lld|mlir|polly)
echo llvm
;;
+ lldb)
+ echo lld
+ ;;
*)
# Nothing to do
;;
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
