tkonolige commented on pull request #10474:
URL: https://github.com/apache/tvm/pull/10474#issuecomment-1059515927


   > Please be aware that default schedule is not always valid on GPU, because 
a hand pick tile size may not fit to a certain GPU architecture.
   
   This is also true of the tophub schedules as they are not specific to a GPU 
model, just target kind.
   
   > IMHO, based on the above discussion this change might be worth to have a 
short RFC to inform the community, as TopHub has been used for a long while and 
we have no clue about the impact. If we are going to apply the change in this 
PR, at least a deprecate warning is necessary.
   
   Do RFCs have the broadest reach? Or would it make more sense to post on the 
forum? I'm not sure who the users of tophub are.
   
   How long should we provide a deprecation warning for? I'd prefer to have 
this fixed as soon as possible instead of waiting. We change a lot of things in 
tvm that are more impactful than this without warning.
   


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to