comaniac commented on a change in pull request #6395:
URL: https://github.com/apache/incubator-tvm/pull/6395#discussion_r492380095
##########
File path: CMakeLists.txt
##########
@@ -76,6 +76,8 @@ tvm_option(USE_COREML "Build with coreml support" OFF)
tvm_option(USE_TARGET_ONNX "Build with ONNX Codegen support" OFF)
tvm_option(USE_ARM_COMPUTE_LIB "Build with Arm Compute Library" OFF)
tvm_option(USE_ARM_COMPUTE_LIB_GRAPH_RUNTIME "Build with Arm Compute Library
graph runtime" OFF)
+tvm_option(USE_TENSORRT "Build with TensorRT" OFF)
Review comment:
The message is a bit confusing. `USE_TENSORRT` means enabling the
TensorRT codegen for graph partitininog. It doesn't require TensorRT to be
available in the system environment. IIUC, maybe it's better to say "Build with
TensorRT codegen", although I just found that "Build with Arm Compute Library"
has the same issue.
@lhutton1 could you also share your thoughts about this?
----------------------------------------------------------------
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:
[email protected]
