[GitHub] [tvm] jverma-quic commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon

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

jverma-quic commented on PR #11539:
URL: https://github.com/apache/tvm/pull/11539#issuecomment-1146202914

   > > Not sure if this is off-topic, but is there a reason we're not using the LLVM version that comes with the Hexagon SDK / Hexagon Toolchain?
   > 
   > I'm not sure if this was discussed earlier. I guess using the LLVM version that comes with the Hexagon Toolchain might work, but one reason I personally would prefer using the upstream release is that the LLVM version that comes with the Toolchain would only be built to support **Hexagon** target and probably a separate LLVM build for ARM/X86 target.
   > 
   > This might be a problem if we ever wish to run tests/models for both `llvm` and `hexagon` target.
   > 
   > Maybe @kparzysz-quic might have a better answer to this question.
   
   @cconvey - In addition to what @quic-sanirudh mentioned above, the Hexagon toolchain in SDK doesn't have fp16 support so it wouldn't work. 


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