[PATCH] D29214: [AMDGPU] Internalize non-kernel symbols

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In https://reviews.llvm.org/D29214#659447, @arsenm wrote:

> Can we move this into clang/ the compiler driver?


It is in the compiler driver, which is in case of OpenCL roccompiler.cpp/rocprogram.cpp passing the option. Otherwise I'm not sure what is to move here. This is target specific thing triggered by the lack of late linking and call support, therefor it is done from the target even if run in opt. It target could request it in the linker that would be even better, but this not the case currently. The moment these conditions change the use of the option and its existence will change as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D29214