junrushao1994 commented on a change in pull request #8767:
URL: https://github.com/apache/tvm/pull/8767#discussion_r691547322
##########
File path: include/tvm/tir/schedule/schedule.h
##########
@@ -219,6 +219,19 @@ class ScheduleNode : public runtime::Object {
* \return The new loops after split
*/
virtual Array<LoopRV> Split(const LoopRV& loop_rv, const
Array<Optional<ExprRV>>& factors) = 0;
+ /*!
+ * \brief Reorder a list of loops. It doesn't require the loops to be
consecutive.
+ * It requires:
+ * 1) The loops are in the same line. That means: the loops can be ordered
to [l_1, l_2, ... ,
+ * l_n] where l_i is an ancestor of l_{i+1} and there are only
single-branch loops between
+ * l_1 and l_n (which also indicates they are under the same scope).
Review comment:
```suggestion
* 1) The loops are in the same line. That means: the loops can be ordered
to [l_1, l_2, ... ,
* l_n] where l_i is the parent of l_{i+1} in the AST, and l_{i + 1}
is always the only child of l_i.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
