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

Commit fe11d2a

Browse files
committed
do not look up the new outer band after shared memory promotion
The code looking up the new outer band was added in the prehistory assuming that some steps following shared memory promotion in makeWithOuterBlockInnerThreadStrategy may need it. This is not and has never been the case. Given that shared memory promotion may now be performed at an arbitrary depth, one can no longer assume any relation between the shared memory promotion copies and the outer band. Remove the code looking up for the outer band.
1 parent 18ace39 commit fe11d2a

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

tc/core/polyhedral/cuda/mapped_scop.cc

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1088,13 +1088,6 @@ std::unique_ptr<MappedScop> MappedScop::makeWithOuterBlockInnerThreadStrategy(
10881088
sharedMemorySize,
10891089
cudaOptions.proto().unroll_copy_shared() &&
10901090
generic.proto.has_unroll());
1091-
1092-
auto bands = ScheduleTree::collectDFSPreorder(
1093-
scop->scheduleRoot(), ScheduleTreeType::Band);
1094-
if (bands.size() == 0) { // Sanity check.
1095-
throw NoBandsException("no bands after promotion");
1096-
}
1097-
outerBand = bands[0];
10981091
}
10991092
}
11001093

0 commit comments

Comments
 (0)