yaxunl added a comment.

If we really want to do this, device lib change and hipcc change need to be 
ready. Since once this is committed without corresponding device lib and hipcc 
change, HIP will break.

On ROCM, device lib is currently installed at /opt/rocm/lib with extension 
.amdgcn.bc. This change requires device libs to be installed to 
/opt/rocm/amdgcn/bitcode and with extension .bc (drop amdgcn in extension). 
Also -device-lib-path option needs to point to /opt/rocm/amdgcn/bitcode or 
dropped.

On Windows, device lib Makefiles need to be changed to drop amdgcn in extension.

Another issue: is /opt/rocm/amdgcn/bitcode the right installation path for 
device lib? Shouldn't it be /opt/rocm/lib/amdgcn/bitcode or 
/opt/rocm/lib/amdgcn ?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77885/new/

https://reviews.llvm.org/D77885



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

Reply via email to