[clang] [llvm] [LLVM][rtsan] Add module pass to initialize rtsan (PR #118989)

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 13 08:11:23 PST 2024


================
@@ -1024,12 +1024,14 @@ void EmitAssemblyHelper::RunOptimizationPipeline(
             FPM.addPass(BoundsCheckingPass());
           });
 
-    if (LangOpts.Sanitize.has(SanitizerKind::Realtime))
+    if (LangOpts.Sanitize.has(SanitizerKind::Realtime)) {
----------------
vitalybuka wrote:

You need to consider performance vs quality.

For other sanitizers it's essentially:
1. If it's early - better precision - in sense we detect issues in C/C++ program which can be optimized out. (can Realtime related issue be optimized out, and do you care about them at all)
2. If it's late - better performance, but we may miss bugs
I believe performance is ~2x difference, so we can't afford that e.g. in Asan.


https://github.com/llvm/llvm-project/pull/118989


More information about the llvm-commits mailing list