kparzysz-quic commented on PR #11116: URL: https://github.com/apache/tvm/pull/11116#issuecomment-1109068204
First of all, feel free to mark cmake version 3.15+ as required for this sub-project. I'm not sure how to answer the question about multiple targets... Both are needed, and that was the easy way to build them. If you want to separate building them, that should be fine, but each target from the same "make" invocation uses the same configuration, so building them together saves some cmake code duplication. We use recursive cmake because (1) components of the same project are built for different targets, and (2) apps are not a part of the TVM core (as is "runtime" for example). You could say that "apps/cpp_rpc" is, and you would be right, which is why I looked into moving it into src/runtime (I'm still planning to propose this, but it's a low priority item at the moment). -- 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]
