Skip to content

Commit 8df7ca0

Browse files
committed
RequirementMachine: Remove useless assertion from MinimalConformances::collectConformanceRules()
We rule out RHS-simplified rules immediately above.
1 parent 32feec7 commit 8df7ca0

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

lib/AST/RequirementMachine/MinimalConformances.cpp

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -417,14 +417,6 @@ void MinimalConformances::collectConformanceRules() {
417417
MutableTerm mutTerm(lhs.begin(), lhs.end() - 2);
418418
assert(!mutTerm.empty());
419419

420-
#ifndef NDEBUG
421-
bool simplified = System.simplify(mutTerm);
422-
// FIXME: Perhaps even if the rule is LHS-simplified, it's parent should be
423-
// canonical?
424-
assert(!simplified || rule.isRHSSimplified());
425-
(void) simplified;
426-
#endif
427-
428420
mutTerm.add(Symbol::forProtocol(parentProto, Context));
429421

430422
// Get a conformance path for X.[P] and record it.

0 commit comments

Comments
 (0)