guansong added a comment.

I am not quite familiar with Clang driver set up, I will add Greg for more 
comments. But I have hacked one the latest YKT tree to support simple AMDGCN 
path the same way as NVPTX. The last patch is here

https://github.com/ROCm-Developer-Tools/hcc2-clang/commit/8c1cce0d39717c9e40ea70aea91e280673de756e

It is not upstreamed but we can compile the same binary for both nvptx and 
amdgcn cards as we designed.

If I understand this correctly, we are now switching nvlink as the default host 
linker whenever nvptx is involved. I am concerned that this may cause trouble 
for integration with other platforms. Maybe this path should be under a special 
option even for nvptx?


Repository:
  rC Clang

https://reviews.llvm.org/D47394



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to