[GitHub] [incubator-tvm] tqchen commented on pull request #5645: [TARGET] add amd_gpu target

2020-05-22 Thread GitBox


tqchen commented on pull request #5645:
URL: https://github.com/apache/incubator-tvm/pull/5645#issuecomment-632731324


   See the example case above(the link to mali log) which uses the opencl, but 
have a separate log by itself



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:
us...@infra.apache.org




[GitHub] [incubator-tvm] tqchen commented on pull request #5645: [TARGET] add amd_gpu target

2020-05-21 Thread GitBox


tqchen commented on pull request #5645:
URL: https://github.com/apache/incubator-tvm/pull/5645#issuecomment-632481759


   We don't need a new target key for autotvm and logs, see the case for mali 
https://github.com/uwsampl/tophub/blob/master/tophub/mali_v0.04.log The device 
is preserved throughout the tvm
   
   In terms of the future runtime data sharing between cpu and GPU. I think 
that it can be solved directly by extending the opencl or vulkan runtime to 
support the shared memory, likely the memory will become vulkan_shared, like 
the cpu_pinned memory in the CUDA.



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:
us...@infra.apache.org