Skip to content

Commit

Permalink
[MLIR] PresburgerSet emptiness check: remove assertions that there ar…
Browse files Browse the repository at this point in the history
…e no symbols

Symbols are now supported in the integer emptiness check. Remove some outdated assertions checking that there are no symbols.

Reviewed By: ftynse

Differential Revision: https://reviews.llvm.org/D100327
  • Loading branch information
Superty committed Apr 12, 2021
1 parent 7714b40 commit 7f9e36b
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions mlir/lib/Analysis/PresburgerSet.cpp
Expand Up @@ -297,7 +297,6 @@ bool PresburgerSet::isEqual(const PresburgerSet &set) const {
/// Return true if all the sets in the union are known to be integer empty,
/// false otherwise.
bool PresburgerSet::isIntegerEmpty() const {
assert(nSym == 0 && "isIntegerEmpty is intended for non-symbolic sets");
// The set is empty iff all of the disjuncts are empty.
for (const FlatAffineConstraints &fac : flatAffineConstraints) {
if (!fac.isIntegerEmpty())
Expand All @@ -307,7 +306,6 @@ bool PresburgerSet::isIntegerEmpty() const {
}

bool PresburgerSet::findIntegerSample(SmallVectorImpl<int64_t> &sample) {
assert(nSym == 0 && "findIntegerSample is intended for non-symbolic sets");
// A sample exists iff any of the disjuncts contains a sample.
for (const FlatAffineConstraints &fac : flatAffineConstraints) {
if (Optional<SmallVector<int64_t, 8>> opt = fac.findIntegerSample()) {
Expand Down

0 comments on commit 7f9e36b

Please sign in to comment.