comaniac commented on pull request #7564:
URL: https://github.com/apache/tvm/pull/7564#issuecomment-804670541


   > We did explore using the same fix as for Ethos-N (giving a 'null' 
ConstantUpdater), but this doesn't work for ACL as ACL does load the constants 
from MetadataModule. Ethos-N just directly serializes the constants into the 
binary module so it doesn't care if they're missing in the MetadataModule.
   
   Could we improve ACL's customized runtime so that it could know if the 
constant should load from MetadataModule or itself? To me, Ethos-N is just an 
extreme case that completely gets rid of constants in MetadataModule, but 
constant updater should be capable of supporting all three cases: all in 
MetadataModule, some in MetadataModule, and none in MetadataModule.


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