Skip to content
This repository was archived by the owner on Apr 28, 2023. It is now read-only.

Commit 30b5bc4

Browse files
author
Sven Verdoolaege
committed
activeDomainPoints: do not take into account mapping filters
In a subsequent commit, mapping filters will no longer be considered to be filters. Change activeDomainPoints first to isolate the effect of this change on activeDomainPoints. There are many calls to activeDomainPoints, so it's not trivial to verify that not including mapping filters is always the right thing to do. However, the included mapping filters are usually seen as a nuisance and if they are really needed in some cases then it's better to take them into account explicitly in those particular cases.
1 parent 6b9bdd3 commit 30b5bc4

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

tc/core/polyhedral/schedule_transforms.cc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ namespace {
105105
/*
106106
* If "node" is any filter, then intersect "domain" with that filter.
107107
*/
108-
isl::union_set applyAnyFilter(isl::union_set domain, const ScheduleTree* node) {
109-
if (auto filterElem = node->elemAsBase<ScheduleTreeElemFilter>()) {
108+
isl::union_set applyFilter(isl::union_set domain, const ScheduleTree* node) {
109+
if (auto filterElem = node->elemAs<ScheduleTreeElemFilter>()) {
110110
return domain.intersect(filterElem->filter_);
111111
}
112112
return domain;
@@ -155,7 +155,7 @@ isl::union_set collectDomain(
155155
isl::union_set activeDomainPointsHelper(
156156
const ScheduleTree* root,
157157
const vector<const ScheduleTree*>& nodes) {
158-
return collectDomain(root, nodes, &applyAnyFilter);
158+
return collectDomain(root, nodes, &applyFilter);
159159
}
160160

161161
} // namespace

0 commit comments

Comments
 (0)