masahi commented on a change in pull request #8366:
URL: https://github.com/apache/tvm/pull/8366#discussion_r668267879
##########
File path: src/tir/transforms/update_pointer_storage_scope.cc
##########
@@ -0,0 +1,91 @@
+/*
Review comment:
Hmm interesting. `attr::storage_scope` and the PointerType should have
the same storage scope, by the invariant at
https://github.com/apache/tvm/blob/3e5e0efe2dfde96f9b3a162ff66524ff79b62723/src/tir/ir/stmt.cc#L64-L73.
A possible explanation is that the value of `scope.tag.length()` at
https://github.com/apache/tvm/blob/main/src/tir/transforms/lower_device_storage_access_info.cc#L70
is different between `main` and this branch. In this case, `GetMemoryInfo`
assumes that a packed function `tvm.info.mem.global.texture` is defined, but I
don't find it anywhere.
Do you have a repro script?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
