jcf94 commented on a change in pull request #6107:
URL: https://github.com/apache/incubator-tvm/pull/6107#discussion_r460388414



##########
File path: src/auto_scheduler/transform_step.cc
##########
@@ -923,5 +958,275 @@ String 
ComputeRootStepNode::PrintAsPythonAPI(Array<te::Stage>* stages,
   return ss.str();
 }
 
+/********** Primitives adding new stages **********/
+
+/*!
+ * \brief Common part for steps that add new stages(e.g. CacheReadStep, 
CacheWriteStep,
+ * RfactorStep). This will return all steps that can change the number of 
stages in a ComputeDAG,
+ * and stop by the current step.
+ */
+Array<Step> GetFormerStageModifiableSteps(Step current_step, const 
Array<Step>& transform_steps) {

Review comment:
       Or `StageMutableSteps` ?
   It seems that just `MutableSteps` is still not quite clear.




----------------------------------------------------------------
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]


Reply via email to