================
@@ -53,7 +53,8 @@ static const SanitizerMask SupportsCoverage =
     SanitizerKind::DataFlow | SanitizerKind::Fuzzer |
     SanitizerKind::FuzzerNoLink | SanitizerKind::FloatDivideByZero |
     SanitizerKind::SafeStack | SanitizerKind::ShadowCallStack |
-    SanitizerKind::Thread | SanitizerKind::ObjCCast | SanitizerKind::KCFI |
+    SanitizerKind::Thread | SanitizerKind::Concurrency |
----------------
yxsamliu wrote:

Since CSan supports sanitizer coverage, coverage makes `needsUbsanRt()` true. 
This can link both `ubsan_standalone` and `csan`, even though CSan already 
contains the coverage support. Could we use only CSan here and add a test? We 
should also handle combinations such as `concurrency,integer`.

https://github.com/llvm/llvm-project/pull/225781
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to