comaniac commented on a change in pull request #6310:
URL: https://github.com/apache/incubator-tvm/pull/6310#discussion_r474084480
##########
File path: src/auto_scheduler/search_policy/sketch_policy_rules.h
##########
@@ -201,6 +201,26 @@ class InitVectorization : public InitPopulationRule {
ResultKind Apply(SketchPolicyNode* policy, State* state) const final;
};
+/********** Mutation **********/
+
+/*! \brief The base class for mutation rules used in the evolutionary search.
*/
+class MutationRule : public InitPopulationRule {
Review comment:
I agree. I'm actually thinking to have a base class for both, because
they are very similar (take a policy and a state and change the state) and it's
weird to define some fields like `ResultKind` twice. Any suggestions?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
