[Driver] Enable ASan on Solaris/SPARC (#107403)
Once PR #107223 lands, ASan can be enabled on Solaris/SPARC. This patch
does just that. As on Solaris/x86, the dynamic ASan runtime lib needs to
be linked with `-z now` to avoid an `AsanInitInternal` cycle.
Tested on `sparcv9-sun-solaris2.11` and `sparc64-unknown-linux-gnu`.
diff --git a/clang/lib/Driver/ToolChains/Solaris.cpp b/clang/lib/Driver/ToolChains/Solaris.cpp
index cf39038..fd3232b 100644
--- a/clang/lib/Driver/ToolChains/Solaris.cpp
+++ b/clang/lib/Driver/ToolChains/Solaris.cpp
@@ -266,8 +266,7 @@
}
}
// Avoid AsanInitInternal cycle, Issue #64126.
- if (ToolChain.getTriple().isX86() && SA.needsSharedRt() &&
- SA.needsAsanRt()) {
+ if (SA.needsSharedRt() && SA.needsAsanRt()) {
CmdArgs.push_back("-z");
CmdArgs.push_back("now");
}
@@ -334,10 +333,11 @@
}
SanitizerMask Solaris::getSupportedSanitizers() const {
+ const bool IsSparc = getTriple().getArch() == llvm::Triple::sparc;
const bool IsX86 = getTriple().getArch() == llvm::Triple::x86;
SanitizerMask Res = ToolChain::getSupportedSanitizers();
- // FIXME: Omit X86_64 until 64-bit support is figured out.
- if (IsX86) {
+ // FIXME: Omit SparcV9 and X86_64 until 64-bit support is figured out.
+ if (IsSparc || IsX86) {
Res |= SanitizerKind::Address;
Res |= SanitizerKind::PointerCompare;
Res |= SanitizerKind::PointerSubtract;