You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/06/02 13:42:35 UTC
[GitHub] [tvm] quic-sanirudh opened a new pull request, #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
quic-sanirudh opened a new pull request, #11539:
URL: https://github.com/apache/tvm/pull/11539
Update to LLVM 14.0.0 release for ci_hexagon docker image.
--
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
[GitHub] [tvm] mehrdadh commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by GitBox <gi...@apache.org>.
mehrdadh commented on PR #11539:
URL: https://github.com/apache/tvm/pull/11539#issuecomment-1146338734
@quic-sanirudh thanks for the PR. LGTM!
I will merge this, note that the image used by CI will be not be automatically updated. You need to send another PR to update it. I can show you how.
--
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
[GitHub] [tvm] mehrdadh merged pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by GitBox <gi...@apache.org>.
mehrdadh merged PR #11539:
URL: https://github.com/apache/tvm/pull/11539
--
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
[GitHub] [tvm] jverma-quic commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by 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
[GitHub] [tvm] quic-sanirudh commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by GitBox <gi...@apache.org>.
quic-sanirudh commented on PR #11539:
URL: https://github.com/apache/tvm/pull/11539#issuecomment-1145076469
> 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.
--
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
[GitHub] [tvm] mehrdadh commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by GitBox <gi...@apache.org>.
mehrdadh commented on PR #11539:
URL: https://github.com/apache/tvm/pull/11539#issuecomment-1146344868
@quic-sanirudh I ran a job to build all docker images and test them: https://ci.tlcpack.ai/job/docker-images-ci/job/daily-docker-image-rebuild/
Once that is finished green, you can open a PR and update the docker tag from this build.
--
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
[GitHub] [tvm] areusch commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by GitBox <gi...@apache.org>.
areusch commented on PR #11539:
URL: https://github.com/apache/tvm/pull/11539#issuecomment-1145003299
@quic-sanirudh i think the CI image build is busted right now due to tflite install. let me see if i can fix that
--
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
[GitHub] [tvm] cconvey commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by GitBox <gi...@apache.org>.
cconvey commented on PR #11539:
URL: https://github.com/apache/tvm/pull/11539#issuecomment-1144996788
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?
--
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
[GitHub] [tvm] quic-sanirudh commented on pull request #11539: [CI] Update to LLVM 14.0.0 for ci_hexagon
Posted by GitBox <gi...@apache.org>.
quic-sanirudh commented on PR #11539:
URL: https://github.com/apache/tvm/pull/11539#issuecomment-1145964185
@areusch @cconvey Can you review and merge this PR when you get a chance. This would enable a bunch of PRs to be rebuilt and merged related to hexagon slice ops.
--
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