[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