[GitHub] [tvm] leandron commented on pull request #7922: [TVMC] add the support of the cross compiler options

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

leandron commented on pull request #7922:
URL: https://github.com/apache/tvm/pull/7922#issuecomment-838775235


   > > > > The change looks OK to me. Apart from my comments below, is there a chance to add a test case for this PR?
   > > > 
   > > > 
   > > > I will do the update of the patch.
   > > > Regarding the test of this PR, I have tested it with the STM32MP157 board. In which location I can add a test case?
   > > > Vincent
   > > 
   > > 
   > > I was referring to unit tests, just to try validate the API changes. The unit tests are currently hosted at - https://github.com/apache/tvm/tree/main/tests/python/driver/tvmc
   > 
   > @leandron how do you execute the tests?
   > On tvm main branch, I tries test_cross_compile_aarch64_tflite_module function and it fails with this error:
   > assert llvm_id != 0, "%s is not an LLVM intrinsic" % name
   > AssertionError: llvm.aarch64.neon.umull is not an LLVM intrinsic
   
   Hi, sorry for the delay. I don't think this error is related to tvmc specifically. It probably has something to do with the way your TVM was built. Which LLVM version are you linking TVM with?


-- 
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