[clang] [rtsan][clang] NFC: Move rtsan init to addSanitizers (PR #119904)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 13 09:22:56 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-codegen
Author: Chris Apple (cjappl)
<details>
<summary>Changes</summary>
This seems to pass tests and behaves as expected.
As suggested by @<!-- -->vitalybuka on #<!-- -->118989
---
Full diff: https://github.com/llvm/llvm-project/pull/119904.diff
1 Files Affected:
- (modified) clang/lib/CodeGen/BackendUtil.cpp (+3-3)
``````````diff
diff --git a/clang/lib/CodeGen/BackendUtil.cpp b/clang/lib/CodeGen/BackendUtil.cpp
index b3728c478772d3..2ef098172c01f5 100644
--- a/clang/lib/CodeGen/BackendUtil.cpp
+++ b/clang/lib/CodeGen/BackendUtil.cpp
@@ -738,6 +738,9 @@ static void addSanitizers(const Triple &TargetTriple,
if (LangOpts.Sanitize.has(SanitizerKind::NumericalStability))
MPM.addPass(NumericalStabilitySanitizerPass());
+ if (LangOpts.Sanitize.has(SanitizerKind::Realtime))
+ MPM.addPass(RealtimeSanitizerPass());
+
auto ASanPass = [&](SanitizerMask Mask, bool CompileKernel) {
if (LangOpts.Sanitize.has(Mask)) {
bool UseGlobalGC = asanUseGlobalsGC(TargetTriple, CodeGenOpts);
@@ -1023,9 +1026,6 @@ void EmitAssemblyHelper::RunOptimizationPipeline(
FPM.addPass(BoundsCheckingPass());
});
- if (LangOpts.Sanitize.has(SanitizerKind::Realtime))
- MPM.addPass(RealtimeSanitizerPass());
-
// Don't add sanitizers if we are here from ThinLTO PostLink. That already
// done on PreLink stage.
if (!IsThinLTOPostLink) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/119904
More information about the cfe-commits
mailing list