[PATCH] D31141: [AMDGPU] Run always inliner early in opt

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 20 11:04:14 PDT 2017


rampitec added a comment.

In https://reviews.llvm.org/D31141#705482, @kzhuravl wrote:

> Not sure which bug you were trying to solve by original change, but moving if we move it "Internalize" then we would have to make function in early-inline.ll amdgpu_kernel. Would that be ok? I added a separate option for now.




In https://reviews.llvm.org/D31141#705482, @kzhuravl wrote:

> Not sure which bug you were trying to solve by original change, but moving if we move it "Internalize" then we would have to make function in early-inline.ll amdgpu_kernel. Would that be ok? I added a separate option for now.




In https://reviews.llvm.org/D31141#705482, @kzhuravl wrote:

> Not sure which bug you were trying to solve by original change, but moving if we move it "Internalize" then we would have to make function in early-inline.ll amdgpu_kernel. Would that be ok? I added a separate option for now.


Yes, changing caller to be a kernel is good.


https://reviews.llvm.org/D31141





More information about the llvm-commits mailing list