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


   @junrushao1994 
   I think thir pr needs 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to