Lunderberg commented on PR #11269: URL: https://github.com/apache/tvm/pull/11269#issuecomment-1123861668
I'd definitely be up for a better name, as `transform_layout_sugared` definitely leaves the reader uncertain what is being done. My one concern with having a completely different name is that it may not be clear to a reader that this is the same underlying schedule primitive in C++. Another alternative would be to merge this as the input handling of `Schedule.transform_layout`. There's already some handling, such as the conversion from `Callable` to `IndexMap`, so this would be similar python-only handling. This would also avoid the issue of having multiple functions that aren't immediately obvious as wrappers around each other. -- 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]
