[GitHub] [tvm] PhilippvK commented on pull request #12534: [Docker][CI][RISC-V] Build riscv-isa-sim (spike) in ci_riscv Docker image to enable RISC-V unit testing

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

PhilippvK commented on PR #12534:
URL: https://github.com/apache/tvm/pull/12534#issuecomment-1239815462

   Here is a short update:
   
   - I dropped the commits related to the `SpikeAOTTestRunner` and will look into testing using the MicroTVM Project API for a future PR. Thanks @areusch and @mehrdadh for letting me know that the currently used AOT test infrastructure is to be deprecated soon.
   - The baremetal toolchain used in the image is now the Xuantie900 version proposed by @alter-xp 
   - For consistency, I refactored the CSI-NN2 insalll script to move the installation of the Linux toolchain to a separate script. As this has a few consequences, I would like to ask @alter-xp to review these changes:
     - Both toolchains are now installed into `/opt/riscv` instead of `/opt/csi-nn2/`
     - The `script/download_toolchain.sh` script is not used anymore. This also means that the installed version of the toolchain has to be kept in sync with the used version of CSI-NN2 by updating the TVM CI script.
     - The used download URLs are hopefully to be replaced with more stable/similar ones once the toolchain will be open sourced
     - After moving the gcc download out of the CSI-NN2 script, if might make sense to do the same with the QEMU installation. What do you think @alter-xp?


-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org