trevor-m opened a new pull request #6553:
URL: https://github.com/apache/incubator-tvm/pull/6553


   When ConvertLayout encounters an operator which is supported by 
ConvertLayout but a desired_layout is not provided by the user, it will create 
a fatal error. With this PR, it will instead just give a warning and leave that 
operator how it was.
   
   Currently, anytime a new operator is added to ConvertLayout, it will break 
ConvertLayout for anyone using it in their code. For example, a BYOC target may 
want all Conv2D ops to be converted to NCHW but doesn't care about the layout 
of other ops. They shouldn't have to specify a desired layout for every single 
op which is supported by ConvertLayout, only the ones which they care about.


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