[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