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

Commit d1e47ff

Browse files
committed
promoteToSharedGreedy: do not intersect schedule with active points
A partial schedule produced by a call to partialSchedule already has its domain intersected with the parent filter nodes. There is no need to separately compute the intersection of parent filters using activeDomainPoints to include it in the schedule. Drop the unnecessary intersection and remove the variable that became unused.
1 parent d02a9e9 commit d1e47ff

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

tc/core/polyhedral/cuda/memory_promotion_heuristic.cc

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -525,11 +525,10 @@ void promoteToSharedGreedy(
525525
// both.
526526
size_t remainingMemory = maxMemory;
527527
for (auto bandNode : bands) {
528-
auto activePoints = activeDomainPoints(root, bandNode);
529528
auto partialSched = partialSchedule(root, bandNode);
530529

531-
auto groupMap = TensorReferenceGroup::accessedWithin(
532-
partialSched.intersect_domain(activePoints), scop.body);
530+
auto groupMap =
531+
TensorReferenceGroup::accessedWithin(partialSched, scop.body);
533532
// Pure affine schedule without (mapping) filters.
534533
auto partialSchedMupa = partialScheduleMupa(root, bandNode);
535534

0 commit comments

Comments
 (0)