[GitHub] [incubator-tvm] comaniac commented on issue #3949: [AutoTVM] Enhance tuning space of split

[GitBox <gi...@apache.org>]

comaniac commented on issue #3949: [AutoTVM] Enhance tuning space of split
URL: https://github.com/apache/incubator-tvm/pull/3949#issuecomment-556842170
 
 
   Yes, this PR also changes the leftmost split factor to -1, but it performs exactly the same behavior as before, because AutoTVM will automatically replace -1 with L/prod(others) when applying the split factor. The reason I made this change is that we are trying to apply a schedule config from one shape to another. For example, assuming we have a config `['tile_x', 'sp', [4, 8]]` from a shape with length 32, and we wish to apply it to another shape with length 16. In this case, `['tile_x', 'sp', [4, 8]]` is inapplicable, but `['tile_x', 'sp', [-1, 8]]` is fine, as it implies `['tile_x', 'sp', [2, 8]]`.
   
   Sorry if this change breaks your schedule. Since CI was passed before merging this PR, I suppose this change should not afffect any schedule in TOPI.
   
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services