junrushao commented on code in PR #13093:
URL: https://github.com/apache/tvm/pull/13093#discussion_r998289205
##########
src/tir/schedule/primitive/cache_read_write.cc:
##########
@@ -75,8 +77,8 @@ struct CacheStageInfo {
Stmt cache_stage;
/*! \brief The map used for ScheduleStateNode::Replace. */
Map<Block, Block> block_reuse;
- /*! \brief A list of blocks that will consume the new cache. */
- Array<StmtSRef> consumer_blocks;
+ /*! \brief A set of blocks that will consume the new cache. */
+ std::unordered_set<StmtSRef, ObjectHash, ObjectEqual> consumer_blocks;
Review Comment:
I was curious if the ordering of `consumer_blocks` might matter, and would
it lead to undeterministic behavior?
--
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]
