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 2020/07/10 17:55:36 UTC

[GitHub] [incubator-tvm] giuseros opened a new pull request #6037: Fix conv2_gemm after target structure update

giuseros opened a new pull request #6037:
URL: https://github.com/apache/incubator-tvm/pull/6037


   After target structure changed in this RFC:
   
   https://discuss.tvm.ai/t/rfc-tvm-target-specification/6844/42
   
   The conv2d optimizations was broken for the following reasons:
   - "target" is now called mtriple (this changes how we test if the
     architecture is AArch64)
   - when we invoke "clang.create_llvm" we still need to specify the
     "--target" option (set to aarch64-linux-gnu)
   
   This submission reverts those changes
   
   Change-Id: I04c597b91ca5800ddf4471255e2a358c60bc048e
   


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

[GitHub] [incubator-tvm] tqchen commented on pull request #6037: Fix conv2_gemm after target structure update

Posted by GitBox <gi...@apache.org>.
tqchen commented on pull request #6037:
URL: https://github.com/apache/incubator-tvm/pull/6037#issuecomment-657941954


   Thanks @giuseros ! CC @junrushao1994 


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

[GitHub] [incubator-tvm] tqchen merged pull request #6037: Fix conv2_gemm after target structure update

Posted by GitBox <gi...@apache.org>.
tqchen merged pull request #6037:
URL: https://github.com/apache/incubator-tvm/pull/6037


   


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

[GitHub] [incubator-tvm] junrushao1994 commented on pull request #6037: Fix conv2_gemm after target structure update

Posted by GitBox <gi...@apache.org>.
junrushao1994 commented on pull request #6037:
URL: https://github.com/apache/incubator-tvm/pull/6037#issuecomment-658340424


   I see. Thank you for the correction!


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