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: [email protected]
