[compiler-rt] [compiler-rt][memprof] Do not add libdl to MEMPROF_UNITTEST_LINK_LIBRARIES (PR #98221)
Alexander Shaposhnikov via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 9 13:56:38 PDT 2024
https://github.com/alexander-shaposhnikov created https://github.com/llvm/llvm-project/pull/98221
The memprof tests do not actually depend on libdl.
They might have a transitive dependency on libdl through the runtime, but the runtime takes care of itself.
Based on the offline discussions it looks like this piece of code in cmake is a historical artifact.
Test plan: ninja check-all
>From e59ace9bb482f8383c9f1b83340131937e79d932 Mon Sep 17 00:00:00 2001
From: Alexander Shaposhnikov <ashaposhnikov at google.com>
Date: Tue, 9 Jul 2024 20:37:39 +0000
Subject: [PATCH] [compiler-rt][memprof] Do not add ldl to
MEMPROF_UNITTEST_LINK_LIBRARIES
---
compiler-rt/lib/memprof/tests/CMakeLists.txt | 1 -
1 file changed, 1 deletion(-)
diff --git a/compiler-rt/lib/memprof/tests/CMakeLists.txt b/compiler-rt/lib/memprof/tests/CMakeLists.txt
index 0b5c302a4ce5d..a35f12bc14265 100644
--- a/compiler-rt/lib/memprof/tests/CMakeLists.txt
+++ b/compiler-rt/lib/memprof/tests/CMakeLists.txt
@@ -49,7 +49,6 @@ endif()
set(MEMPROF_UNITTEST_LINK_LIBRARIES
${COMPILER_RT_UNWINDER_LINK_LIBS}
${SANITIZER_TEST_CXX_LIBRARIES})
-append_list_if(COMPILER_RT_HAS_LIBDL -ldl MEMPROF_UNITTEST_LINK_LIBRARIES)
# Adds memprof tests for each architecture.
macro(add_memprof_tests_for_arch arch)
More information about the llvm-commits
mailing list