[PATCH] D117384: [Polly] Swapping the order makes the assert work

[QingFu Wei via Phabricator]

yzwqf created this revision.
yzwqf added a reviewer: pollydev.
Herald added a reviewer: bollu.
yzwqf requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

at before, assert statement within the "case isl_schedule_node_set:" is apparently unreachable.


Repository:
rG LLVM Github Monorepo

https://reviews.llvm.org/D117384

Files:
polly/include/polly/ScheduleTreeTransform.h


Index: polly/include/polly/ScheduleTreeTransform.h
===================================================================
--- polly/include/polly/ScheduleTreeTransform.h
+++ polly/include/polly/ScheduleTreeTransform.h
@@ -47,9 +47,9 @@
return getDerived().visitSequence(Node.as<isl::schedule_node_sequence>(),
std::forward<Args>(args)...);
case isl_schedule_node_set:
+ assert(isl_schedule_node_n_children(Node.get()) >= 2);
return getDerived().visitSet(Node.as<isl::schedule_node_set>(),
std::forward<Args>(args)...);
- assert(isl_schedule_node_n_children(Node.get()) >= 2);
case isl_schedule_node_leaf:
assert(isl_schedule_node_n_children(Node.get()) == 0);
return getDerived().visitLeaf(Node.as<isl::schedule_node_leaf>(),