mbrookhart commented on a change in pull request #7611:
URL: https://github.com/apache/tvm/pull/7611#discussion_r590506226
##########
File path: src/tir/transforms/storage_access.cc
##########
@@ -132,6 +132,7 @@ void StorageAccessVisitor::VisitStmt_(const AttrStmtNode*
op) {
StmtExprVisitor::VisitStmt_(op);
}
env_threads_.pop_back();
+ } else if (op->attr_key == attr::hand_threaded) {
} else {
Review comment:
I'll add a comment, but the point of this is to skip thread planning for
the enclosed block, i.e., not to visit the statements in this block. I do that
because 1) it's already threaded and 2) the thread planning pass can't handle
the per-thread read/write/sync I'm doing.
----------------------------------------------------------------
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:
[email protected]
