wrongtest commented on a change in pull request #8338:
URL: https://github.com/apache/tvm/pull/8338#discussion_r663608412



##########
File path: src/tir/transforms/storage_rewrite.cc
##########
@@ -504,8 +503,13 @@ class StoragePlanRewriter : public StmtExprMutator {
     return MergeNest(nest, body);
   }
   // Remap the index
-  PrimExpr RemapIndex(DataType dtype, PrimExpr index, StorageEntry* e) {
+  PrimExpr RemapIndex(PrimExpr index, StorageEntry* e) {

Review comment:
       Yes you're right! I checked some mixed dtypes and found that the type 
casting is done before indexing for load/store. I repush the change which 
modify just one line of code about lanes, can you kindly review it again?




-- 
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: commits-unsubscr...@tvm.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to