[clang] Allow multiple sanitizers on baremetal targets. (PR #72933)
Fangrui Song via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 20 22:10:46 PST 2023
================
@@ -491,3 +491,26 @@ void baremetal::Linker::ConstructJob(Compilation &C, const JobAction &JA,
JA, *this, ResponseFileSupport::AtFileCurCP(),
Args.MakeArgString(TC.GetLinkerPath()), CmdArgs, Inputs, Output));
}
+
+SanitizerMask BareMetal::getSupportedSanitizers() const {
+ const bool IsX86_64 = getTriple().getArch() == llvm::Triple::x86_64;
----------------
MaskRay wrote:
Add a comment suggesting that this allows a superset of other targets allow so that it is clearer if someone intends to extend the list in the future (say. when we get a new sanitizer).
https://github.com/llvm/llvm-project/pull/72933
More information about the cfe-commits
mailing list