anijain2305 commented on issue #4249: [TOPI][AlterOpLayout][ARM] Enabling NHWC to NCHW layout transformation. URL: https://github.com/apache/incubator-tvm/pull/4249#issuecomment-552222356 > Thanks for your detailed explanation @anijain2305 ! I missed you change of removing legalization. > > AFAIK, AlterOpLayout is not always enabled for different _opt level_, in which case the NHWC computing may encounter build failure. Not sure AlterOpLayout is a good solution for all cases. That is a good point. But, even Legalize is enabled at opt_level >= 1. AlterOpLayout is enabled at opt_level = 3. So, I think this is ok. As tq said, I think we should work on the Explicit layout Convertor Pass to solve the opt_level problem separately. I have some thoughts on that and take that task up.
---------------------------------------------------------------- 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] With regards, Apache Git Services
