comaniac opened a new pull request #7232:
URL: https://github.com/apache/tvm/pull/7232


   #7018 introduces the dynamic support to `strided_slice`. However, in the 
following case, the static workload will be treat as a dynamic:
   
   ```
   cpp.strided_slice(weight, begin=[0, 0, 0, 0], end=[None, None, filter_h, 
filter_w])
   ```
   
   This is because the `None` in `end` is not `IntImmNode` and will be judged 
as a non-constant array. This PR changes the behavior by ignoring the undefined 
elements.
   
   In addition, the dynamic path also assumes `strides` array is always given, 
but it will be initialized to array of 1 after this path. Since I'm not sure 
what's the right behavior, I only add a checker to error out.
   
   cc @kevinthesun @mbrookhart 


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