Skip to content

Conversation

@brad0
Copy link
Contributor

@brad0 brad0 commented Jan 31, 2025

No description provided.

@llvmbot llvmbot added clang Clang issues not falling into any other category clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' labels Jan 31, 2025
@llvmbot
Copy link
Member

llvmbot commented Jan 31, 2025

@llvm/pr-subscribers-clang-driver

@llvm/pr-subscribers-clang

Author: Brad Smith (brad0)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/125179.diff

1 Files Affected:

  • (modified) clang/lib/Driver/ToolChains/CommonArgs.cpp (+2-1)
diff --git a/clang/lib/Driver/ToolChains/CommonArgs.cpp b/clang/lib/Driver/ToolChains/CommonArgs.cpp
index 2c4b082bcce4a6..6ec19dbb89403c 100644
--- a/clang/lib/Driver/ToolChains/CommonArgs.cpp
+++ b/clang/lib/Driver/ToolChains/CommonArgs.cpp
@@ -1446,7 +1446,8 @@ void tools::linkSanitizerRuntimeDeps(const ToolChain &TC,
   // Required for backtrace on some OSes
   if (TC.getTriple().isOSFreeBSD() ||
       TC.getTriple().isOSNetBSD() ||
-      TC.getTriple().isOSOpenBSD())
+      TC.getTriple().isOSOpenBSD() ||
+      TC.getTriple().isOSDragonFly())
     CmdArgs.push_back("-lexecinfo");
   // There is no libresolv on Android, FreeBSD, OpenBSD, etc. On musl
   // libresolv.a, even if exists, is an empty archive to satisfy POSIX -lresolv

@github-actions
Copy link

github-actions bot commented Jan 31, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

@brad0 brad0 force-pushed the clang_dragonfly_execinfo branch from b3a80b3 to 984c3bb Compare January 31, 2025 07:24
@brad0 brad0 force-pushed the clang_dragonfly_execinfo branch from 984c3bb to 3064235 Compare January 31, 2025 07:32
@brad0 brad0 changed the title [Driver] DragonFly also needs libexecinfo [Driver] Add DragonFly for handling of libdl and libexecinfo Jan 31, 2025
@brad0 brad0 merged commit bc47dae into llvm:main Jan 31, 2025
8 checks passed
@brad0 brad0 deleted the clang_dragonfly_execinfo branch January 31, 2025 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants