tvm-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-tvm] FrozenGene commented on pull request #5754: [RFC] Improve quantized convolution performance for armv8 architectures
Date Thu, 11 Jun 2020 12:08:06 GMT

FrozenGene commented on pull request #5754:
URL: https://github.com/apache/incubator-tvm/pull/5754#issuecomment-642601817


   > Hi @FrozenGene ,
   > 
   > The idea of adding the algorithm name to the attributes would work if the legalization
step was run after we pick the strategy. It is instead run before, so it is unaware of the
strategy picked. 
   > 
   > 
   > 
   > Maybe we could add a new pass that runs based on the strategy? Or we can hack in `_alter_conv2d_layout`?

   > 
   > 
   
   @giuseros what you mean run based on the strategy?
   
   in alter_op_layout, we could extract workload[0] to get strategy, however could you help
me to double check whether our autotvm tuning will use alter_op_layout pass?(i.e. O3), I have
forgot a little bit. If so, maybe we could change the dtype here according to strategy. cc
@anijain2305 any better idea too?
   


----------------------------------------------------------------
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:
users@infra.apache.org



Mime
View raw message