huajsj commented on a change in pull request #10543:
URL: https://github.com/apache/tvm/pull/10543#discussion_r825381312
##########
File path: src/runtime/pipeline/pipeline_struct.h
##########
@@ -96,24 +128,21 @@ class DataNotify {
bool data_ready_ = false;
/*!\brief Whether the thread should exit or not.*/
std::atomic<bool> exit_state_{false};
- /*!
- * \brief The 'ModuleInterfaceID' in which the data was ready and triggered
this
- * notification.
- */
+ /*!\brief The 'ModuleInterfaceID' of an interface which sent this
notification.*/
ModuleInterfaceID notification_source_;
public:
/*!
* \brief Constructing the DataNotify class.
- * \param parent_output_id The id of a runtime interface which is sending
out the data
+ * \param source_interface_id The id of a runtime interface which sent out
the data
* notification.
*/
- explicit DataNotify(ModuleInterfaceID parent_output_id) {
- notification_source_ = parent_output_id;
+ explicit DataNotify(ModuleInterfaceID source_interface_id) {
+ notification_source_ = source_interface_id;
}
/*!
- * \brief Getting the notification source.
- * \return The first 'int' is the runtime index, and the second 'int' is the
output index.
+ * \brief Getting the notification target.
+ * \return The ID of the interface which sent out the nofication.
Review comment:
fixed.
--
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]
