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

Commit fe139b6

Browse files
author
Sven Verdoolaege
committed
tightenLaunchBounds: check that at least some paths in the tree are mapped
This is just a basic sanity check. Checking that all paths are mapped would require special cases for nodes such as synchronizations that should not be (explicitly) mapped to threads.
1 parent 54017c6 commit fe139b6

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tc/core/polyhedral/cuda/tighten_launch_bounds.cc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,7 @@ size_t maxValue(const Scop& scop, const MappingIdType& id) {
8989
<< "Opportunity for tightening launch bounds with shifting -> min:"
9090
<< min;
9191
TC_CHECK(max < sizetMax) << "missing mapping to " << id << "\n" << *root;
92+
TC_CHECK(min < sizetMax) << "missing mapping to " << id << " type\n" << *root;
9293
// Inclusive range needs + 1 to translate to sizes
9394
return max + 1;
9495
}

0 commit comments

Comments
 (0)