Skip to content

Commit f750796

Browse files
artagnongithub-actions[bot]
authored andcommitted
Automerge: SLPVectorizer: strip bad FIXME (NFC) (#122888)
Follow up on 4a0d53a (PatternMatch: migrate to CmpPredicate) to get rid of the FIXME it introduced in SLPVectorizer: the FIXME is bad, and we'd get no testable impact by using CmpPredicate::getMatching here.
2 parents 3c319d4 + 0fe8469 commit f750796

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11529,7 +11529,6 @@ BoUpSLP::getEntryCost(const TreeEntry *E, ArrayRef<Value *> VectorizedVals,
1152911529
? CmpInst::BAD_FCMP_PREDICATE
1153011530
: CmpInst::BAD_ICMP_PREDICATE;
1153111531
auto MatchCmp = m_Cmp(CurrentPred, m_Value(), m_Value());
11532-
// FIXME: Use CmpPredicate::getMatching here.
1153311532
if ((!match(VI, m_Select(MatchCmp, m_Value(), m_Value())) &&
1153411533
!match(VI, MatchCmp)) ||
1153511534
(CurrentPred != static_cast<CmpInst::Predicate>(VecPred) &&

0 commit comments

Comments
 (0)