[GitHub] [incubator-tvm] leandron commented on a change in pull request #6170: [BYOC][ACL] Improve installation tutorial

[GitBox <gi...@apache.org>]

leandron commented on a change in pull request #6170:
URL: https://github.com/apache/incubator-tvm/pull/6170#discussion_r462844056



##########
File path: docs/deploy/arm_compute_lib.rst
##########
@@ -42,6 +60,19 @@ to compile an ACL module on an x86 machine and then run the module on a remote A
 need to use USE_ARM_COMPUTE_LIB=ON on the x86 machine and USE_ARM_COMPUTE_LIB_GRAPH_RUNTIME=ON on the remote
 AArch64 device.
 
+Using USE_ARM_COMPUTE_LIB_GRAPH_RUNTIME=ON will mean that ACL binaries are searched for by cmake in the
+default locations (see https://cmake.org/cmake/help/v3.4/command/find_library.html). In addition to this,
+/path-to-tvm-project/acl/ will also be searched. It is likely that you will need to set your own path to
+locate ACL. This can be done by specifying a path in the place of ON.
+
+These flags should be set in your config.cmake file. For example:

Review comment:
       Minor suggestion: might worth reinforcing here that default value for `USE_ARM_COMPUTE_LIB` is `OFF`.




----------------------------------------------------------------
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:
users@infra.apache.org