[llvm] [LLVM][rtsan] rtsan transform to preserve CFAnalyes (PR #102651)

via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 9 10:49:57 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Chris Apple (cjappl)

<details>
<summary>Changes</summary>

Follow on to #<!-- -->101232, as suggested in the comments, narrow the scope of the preserved analyses.

---
Full diff: https://github.com/llvm/llvm-project/pull/102651.diff


1 Files Affected:

- (modified) llvm/lib/Transforms/Instrumentation/RealtimeSanitizer.cpp (+4-1) 


``````````diff
diff --git a/llvm/lib/Transforms/Instrumentation/RealtimeSanitizer.cpp b/llvm/lib/Transforms/Instrumentation/RealtimeSanitizer.cpp
index 5663f446613b5..7854cf4f2c625 100644
--- a/llvm/lib/Transforms/Instrumentation/RealtimeSanitizer.cpp
+++ b/llvm/lib/Transforms/Instrumentation/RealtimeSanitizer.cpp
@@ -53,7 +53,10 @@ PreservedAnalyses RealtimeSanitizerPass::run(Function &F,
   if (F.hasFnAttribute(Attribute::SanitizeRealtime)) {
     insertCallAtFunctionEntryPoint(F, "__rtsan_realtime_enter");
     insertCallAtAllFunctionExitPoints(F, "__rtsan_realtime_exit");
-    return PreservedAnalyses::none();
+
+    PreservedAnalyses PA;
+    PA.preserveSet<CFGAnalyses>();
+    return PA;
   }
 
   return PreservedAnalyses::all();

``````````

</details>


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


More information about the llvm-commits mailing list