[PATCH] D36849: [AMDGPU] Port of HSAIL inliner

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 23 23:07:00 PDT 2017


rampitec added a comment.

In https://reviews.llvm.org/D36849#851079, @arsenm wrote:

> I don't like the idea of having a custom inlined, and thinking about this is very premature. We haven't attempted any benchmarking of calls or looked at current inlining behavior or if it could be improved. I would rather not commit this until it is clear it is absolutely necessary.


We did a lot of benchmarking with HSAIL. LLVM general inliner did not change much since then and does not factor stuff important for us.


https://reviews.llvm.org/D36849





More information about the llvm-commits mailing list