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 2021/11/25 08:24:20 UTC

[GitHub] [tvm] ZQPei edited a comment on pull request #9585: [BugFix] fix nvptx not supported by device_enabled error

ZQPei edited a comment on pull request #9585:
URL: https://github.com/apache/tvm/pull/9585#issuecomment-978942038


   @junrushao1994 
   I think thir pr needs a further review, because another error occurs in ci https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-9585/1/pipeline/267.
   
   Quoted error is
   > E             File "/workspace/src/target/llvm/codegen_nvptx.cc", line 146
   E           TVMError: Do not support sync shared.dyn
   
   This error is raised because of `shared.dyn` not supported by "nvptx" target.
   And it was previously covered by `device_enabled("nvptx") returning False` in the last few months.
   
   https://github.com/apache/tvm/blob/0a4cc89faf85c474df15bf3ba82ccac89e785fdc/tests/python/unittest/test_tir_ir_builder.py#L499-L552
   
   I have tried to fix it by this commit https://github.com/apache/tvm/pull/9585/commits/a8d14b1890709df8fa3b20a40ea6ac3f54cfb060, and it works fine on my 2080ti gpu card.
   But I am not sure if it is really OK.


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