[GitHub] [tvm] manupa-arm commented on a change in pull request #10344: [microNPU] Integrate rolling buffers in Arm(R) Ethos(TM)-U

[GitBox <gi...@apache.org>]

manupa-arm commented on a change in pull request #10344:
URL: https://github.com/apache/tvm/pull/10344#discussion_r814741231



##########
File path: python/tvm/relay/backend/contrib/ethosu/codegen.py
##########
@@ -31,6 +31,12 @@
 from tvm.relay.backend.contrib.ethosu.op import op_attrs
 from tvm.relay.backend.contrib.ethosu import op
 
+# We are currently using copy_constants scheduler In the long run,
+# this should be a single intelligent and a composite scheduler
+# that can perform scheduling based on user inputs such as
+# scratch memory size.
+CASCADER = copy_constants()

Review comment:
       nit : let us use something like SCHEDULER -- because cascading is a way of scheduling and copy_constants is not a cascader scheduler.




-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org