[GitHub] [tvm] hypercubestart commented on a change in pull request #7831: [AutoTVM] [TOPI] Support AutoTVM for int4 tensorcore

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

hypercubestart commented on a change in pull request #7831:
URL: https://github.com/apache/tvm/pull/7831#discussion_r612066255



##########
File path: python/tvm/topi/cuda/conv2d_hwnc_tensorcore.py
##########
@@ -254,13 +253,8 @@ def schedule_hwnc_tensorcore_cuda(cfg, s, Conv):
     vector_as = cfg["vector_as"].val
     vector_ws = cfg["vector_ws"].val
     split_block_k_nums = cfg["split_block_k_nums"].val
-    fuse_pack = cfg["fuse_pack"].val
 
-    if not fuse_pack:
-        s[packed_data].compute_inline()
-    else:
-        with Target("cuda"):
-            schedule_injective_from_existing(s, packed_data)
+    s[packed_data].compute_inline()

Review comment:
       having `fuse` and `compute_at` in the search space causes loop itervars to touch buffers a different number of times, which causes errors for AutoTVM's feature extraction, i.e feature extraction will return different size features




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