tqchen commented on a change in pull request #5962:
URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r454058968
##########
File path: src/auto_schedule/utils.h
##########
@@ -184,22 +184,36 @@ inline void PrintTitle(const std::string& title, int
verbose) {
}
/*!
- * \brief A simple thread pool.
+ * \brief A simple thread pool to perform parallel for.
* TODO(merrymercy): Move this to `src/support/parallel_for`
*/
-class ThreadPool {
+class ParallelFor {
Review comment:
@jcf94 Sorry I wasn't meant to say that we should rename ThreadPool to
ParallelFor, instead we should hide the use of threadpool behind a parallel_for
API, in similar style to
https://docs.microsoft.com/en-us/cpp/parallel/concrt/parallel-algorithms?view=vs-2019#parallel_for
----------------------------------------------------------------
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:
us...@infra.apache.org
