llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Vitaly Buka (vitalybuka) <details> <summary>Changes</summary> Show why we don't need regular CFI runtime, if CFI diag is linked. --- Full diff: https://github.com/llvm/llvm-project/pull/135881.diff 1 Files Affected: - (modified) clang/lib/Driver/SanitizerArgs.cpp (+1-2) ``````````diff diff --git a/clang/lib/Driver/SanitizerArgs.cpp b/clang/lib/Driver/SanitizerArgs.cpp index 3c7cd562a14e3..1db9da1b60939 100644 --- a/clang/lib/Driver/SanitizerArgs.cpp +++ b/clang/lib/Driver/SanitizerArgs.cpp @@ -371,8 +371,7 @@ bool SanitizerArgs::needsUbsanCXXRt() const { } bool SanitizerArgs::needsCfiRt() const { - return !(Sanitizers.Mask & SanitizerKind::CFI & ~TrapSanitizers.Mask) && - CfiCrossDso && !ImplicitCfiRuntime; + return !needsCfiDiagRt() && CfiCrossDso && !ImplicitCfiRuntime; } bool SanitizerArgs::needsCfiDiagRt() const { `````````` </details> https://github.com/llvm/llvm-project/pull/135881 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
