[llvm] [LowerAllowCheck] Rename removeUbsanTrap() to lowerAllowCheck() (PR #149847)
Marco Elver via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 21 09:43:39 PDT 2025
https://github.com/melver created https://github.com/llvm/llvm-project/pull/149847
No traps are removed directly nor is this restricted to UBSan, therefore rename the function doing the transformation of the intrinsic to match its intent.
NFC.
>From 6e43bcc54e540bd02812e683267738c2ea12ef60 Mon Sep 17 00:00:00 2001
From: Marco Elver <elver at google.com>
Date: Mon, 21 Jul 2025 18:41:17 +0200
Subject: [PATCH] [LowerAllowCheck] Rename removeUbsanTrap() to
lowerAllowCheck()
No traps are removed directly nor is this restricted to UBSan, therefore
rename the function doing the transformation of the intrinsic to match
its intent.
NFC.
---
llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp b/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp
index 55f3239ada100..2486e77ab0137 100644
--- a/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp
+++ b/llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp
@@ -72,7 +72,7 @@ static void emitRemark(IntrinsicInst *II, OptimizationRemarkEmitter &ORE,
}
}
-static bool removeUbsanTraps(Function &F, const BlockFrequencyInfo &BFI,
+static bool lowerAllowChecks(Function &F, const BlockFrequencyInfo &BFI,
const ProfileSummaryInfo *PSI,
OptimizationRemarkEmitter &ORE,
const LowerAllowCheckPass::Options &Opts) {
@@ -160,7 +160,7 @@ PreservedAnalyses LowerAllowCheckPass::run(Function &F,
OptimizationRemarkEmitter &ORE =
AM.getResult<OptimizationRemarkEmitterAnalysis>(F);
- return removeUbsanTraps(F, BFI, PSI, ORE, Opts)
+ return lowerAllowChecks(F, BFI, PSI, ORE, Opts)
// We do not change the CFG, we only replace the intrinsics with
// true or false.
? PreservedAnalyses::none().preserveSet<CFGAnalyses>()
More information about the llvm-commits
mailing list