You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/12/04 10:52:21 UTC

[GitHub] [tvm] hzfan commented on issue #6691: [Performance] Performance regression with int64 indices INDEX_DEFAULT_I64=ON (PR #6143)

hzfan commented on issue #6691:
URL: https://github.com/apache/tvm/issues/6691#issuecomment-738715741


   @tqchen The cast originates from https://github.com/apache/tvm/blob/main/src/te/schedule/operation_inline.cc#L67
   For example, before inline,
   ```
   for i from 0i32 to 10i32
     for j from 0i32 to 10i32
       T_add[i, j] = placeholder[i + j + 1i32, j]
   
   for i from 0i64 to 10i64
     for j from 0i64 to 10i64
       result[i, j] = T_add[i, j]
   ```
   
   after inline,
   
   ```
   result[i, j] = placeholder[cast(i32, i) + cast(i32, j) + 1i32, cast(i32, j)]
   ```
   
   Perhaps I can do something with the SumExpr in CanonicalSimplifier to simplify this case.


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