[PATCH] D51434: [HIP] Add -amdgpu-internalize-symbols option to opt

[Yaxun Liu via Phabricator via cfe-commits]

yaxunl added a comment.

In https://reviews.llvm.org/D51434#1217971, @arsenm wrote:

> https://reviews.llvm.org/D51209 is the patch. I think HIP will need an additional patch, since I think it isn’t subclassing the amdgpu toolchain


Yes since HIP has different toolchain. This does not affect kernel's visibility, right?


https://reviews.llvm.org/D51434