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

[QingFu Wei via Phabricator via llvm-commits]

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>(),


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D117384.400236.patch
Type: text/x-patch
Size: 848 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220115/2e63037d/attachment.bin>