csullivan commented on a change in pull request #9343:
URL: https://github.com/apache/tvm/pull/9343#discussion_r733974649
##########
File path: apps/hexagon_launcher/README.md
##########
@@ -40,29 +40,32 @@ tvm_runtime, as well as the Hexagon launcher shared library
and its correspondin
tvm_runtime. As described in the [Manual compilation](#Manual compilation)
section
each component requires Hexagon and android dependencies. When building the
launcher
along with TVM these configurations must be providing when invoking cmake. A
minimal
-example invocation for compiling TVM along with the Hexagon launcher is
included below,
+example invocation for compiling TVM along with the Hexagon launcher is
included below:
```
-cmake -DCMAKE_MAKE_PROGRAM=make \
- -DCMAKE_C_COMPILER=clang \
- -DCMAKE_CXX_COMPILER=clang++ \
+cmake -DCMAKE_C_COMPILER=/path/to/clang \
+ -DCMAKE_CXX_COMPILER=/path/to/clang++ \
-DCMAKE_CXX_FLAGS='-stdlib=libc++' \
-DCMAKE_CXX_STANDARD=14 \
- -DUSE_LLVM=/path/to/hexagon/llvm/bin/llvm-config \
+ -DUSE_LLVM=/path/to/llvm/bin/llvm-config \
Review comment:
I had the `/path/to/hexagon` prefix to indicate the llvm libs need to
have hexagon codegen support. Perhaps this would be worth noting in the text
below.
--
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]
