[GitHub] [tvm] sergey-grovety commented on pull request #8990: [microTVM] Update support for ARMv7m intrinsic

2021-09-28 Thread GitBox
sergey-grovety commented on pull request #8990: URL: https://github.com/apache/tvm/pull/8990#issuecomment-929345342 > not sure if we should land #9092 first and refactor test logic there? otherwise aside from the two comments i think we can merge and move forwards. Yes. Need to clean

[GitHub] [tvm] sergey-grovety commented on pull request #8990: [microTVM] Update support for ARMv7m intrinsic

2021-09-28 Thread GitBox
sergey-grovety commented on pull request #8990: URL: https://github.com/apache/tvm/pull/8990#issuecomment-929345342 > not sure if we should land #9092 first and refactor test logic there? otherwise aside from the two comments i think we can merge and move forwards. Yes. Need to clean

[GitHub] [tvm] sergey-grovety commented on pull request #8990: [microTVM] Update support for ARMv7m intrinsic

2021-09-22 Thread GitBox
sergey-grovety commented on pull request #8990: URL: https://github.com/apache/tvm/pull/8990#issuecomment-925143913 > @sergey-grovety i chatted with @u99127 and he indicated his preference is to use the ARM FVP to test these schedules. those platforms should properly support SMLAD intrinsi

[GitHub] [tvm] sergey-grovety commented on pull request #8990: [microTVM] Update support for ARMv7m intrinsic

2021-09-20 Thread GitBox
sergey-grovety commented on pull request #8990: URL: https://github.com/apache/tvm/pull/8990#issuecomment-923092946 > just a heads up about this PR: #9026 > I added a test_utils.py in this PR for other reasons and after merge you need to rebase with main to avoid merge conflict. O

[GitHub] [tvm] sergey-grovety commented on pull request #8990: [microTVM] Update support for ARMv7m intrinsic

2021-09-15 Thread GitBox
sergey-grovety commented on pull request #8990: URL: https://github.com/apache/tvm/pull/8990#issuecomment-920335684 > for the test image maybe you could reuse this image: https://github.com/apache/tvm/blob/main/tests/micro/testdata/mnist/digit-2.jpg Yes, sure. -- This is an automa