mbs-octoml commented on code in PR #11173:
URL: https://github.com/apache/tvm/pull/11173#discussion_r865121058
##########
include/tvm/target/compilation_config.h:
##########
@@ -74,16 +63,37 @@ class CompilationConfigNode : public Object {
Target host_target;
/*!
- * \brief Vector of all available \p Targets for compiling primitive
operators. May contain
- * a \p Target for the same device type as for the \p host_target, however
the \p host_target
- * should be used for all host computations and data. Each \p Target will
have \p host_target
- * as its host.
+ * \brief Vector of all available \p Targets for partitioning or compiling
primitive tensor
+ * operators (kernels). May contain a \p Target for the same device type as
for the
+ * \p host_target, however the \p host_target should be used for all host
computations and data.
+ * Each \p Target will have \p host_target as its 'host'.
+ *
+ * It is possible to have multiple primitive targets for the same device
type. However given
Review Comment:
I added a TODO to revisit if this is tripping folks up at the various build
apis.
--
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]
