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

2020-10-29 Thread GitBox
tqchen commented on issue #6691: URL: https://github.com/apache/incubator-tvm/issues/6691#issuecomment-719100370 ping @hzfan can you please follow a bit further ? It is a great chance for us to improve the simplifcation and i64 flow

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

2020-10-28 Thread GitBox
tqchen commented on issue #6691: URL: https://github.com/apache/incubator-tvm/issues/6691#issuecomment-718038395 Fixed by #6771. However, it will still be great to followup on this case This is an automated message from the

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

2020-10-15 Thread GitBox
tqchen commented on issue #6691: URL: https://github.com/apache/incubator-tvm/issues/6691#issuecomment-709583351 Thanks @trevor-m cc @hzfan Given that those are constant shapes, we should expect NarrowDataTypes to narrow dtypes to i32, it would be great to look into the IR those kernels