[llvm] [TLI] Remove leftover assert in TargetLibraryInfoImpl initialize (PR #79056)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 22 12:55:46 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-analysis
Author: None (tltao)
<details>
<summary>Changes</summary>
Remove unnecessary assert for a sorted StandardNames after implementation of getLibFunc is changed from binary search to a DenseMap Lookup in commit 7d950f040e3da66ec83f91b58d8c2220483d6335.
---
Full diff: https://github.com/llvm/llvm-project/pull/79056.diff
1 Files Affected:
- (modified) llvm/lib/Analysis/TargetLibraryInfo.cpp (-6)
``````````diff
diff --git a/llvm/lib/Analysis/TargetLibraryInfo.cpp b/llvm/lib/Analysis/TargetLibraryInfo.cpp
index 58749e559040a74..25951d2a7fe63c5 100644
--- a/llvm/lib/Analysis/TargetLibraryInfo.cpp
+++ b/llvm/lib/Analysis/TargetLibraryInfo.cpp
@@ -163,12 +163,6 @@ bool TargetLibraryInfoImpl::isCallingConvCCompatible(Function *F) {
/// triple gets a sane set of defaults.
static void initialize(TargetLibraryInfoImpl &TLI, const Triple &T,
ArrayRef<StringLiteral> StandardNames) {
- // Verify that the StandardNames array is in alphabetical order.
- assert(
- llvm::is_sorted(StandardNames,
- [](StringRef LHS, StringRef RHS) { return LHS < RHS; }) &&
- "TargetLibraryInfoImpl function names must be sorted");
-
// Set IO unlocked variants as unavailable
// Set them as available per system below
TLI.setUnavailable(LibFunc_getc_unlocked);
``````````
</details>
https://github.com/llvm/llvm-project/pull/79056
More information about the llvm-commits
mailing list