Skip to content

Commit 36089e5

Browse files
authored
[LowerAllowCheck] Rename removeUbsanTrap() to lowerAllowChecks() (llvm#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.
1 parent d9527be commit 36089e5

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ static void emitRemark(IntrinsicInst *II, OptimizationRemarkEmitter &ORE,
7272
}
7373
}
7474

75-
static bool removeUbsanTraps(Function &F, const BlockFrequencyInfo &BFI,
75+
static bool lowerAllowChecks(Function &F, const BlockFrequencyInfo &BFI,
7676
const ProfileSummaryInfo *PSI,
7777
OptimizationRemarkEmitter &ORE,
7878
const LowerAllowCheckPass::Options &Opts) {
@@ -160,7 +160,7 @@ PreservedAnalyses LowerAllowCheckPass::run(Function &F,
160160
OptimizationRemarkEmitter &ORE =
161161
AM.getResult<OptimizationRemarkEmitterAnalysis>(F);
162162

163-
return removeUbsanTraps(F, BFI, PSI, ORE, Opts)
163+
return lowerAllowChecks(F, BFI, PSI, ORE, Opts)
164164
// We do not change the CFG, we only replace the intrinsics with
165165
// true or false.
166166
? PreservedAnalyses::none().preserveSet<CFGAnalyses>()

0 commit comments

Comments
 (0)