[PATCH] D66430: [AMDGPU] Kernel calls lowering

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In D66430#1635654 <https://reviews.llvm.org/D66430#1635654>, @arsenm wrote:

> This pass should be removed. Clang needs to emit a separate function body up-front. I want the IR verifier to reject all calls to kernels. Since a module may need to export a callable function with the name of the kernel, this has a name mangling impact that the frontend needs to take care of


I agree in general. However, we do not support late linking so far but do support OpenCL. Is there any reason to keep this pass even as a temporary solution in the private branch?


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

https://reviews.llvm.org/D66430