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

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 16 10:03:07 PDT 2017


rampitec added a comment.

In https://reviews.llvm.org/D31016#702818, @arsenm wrote:

> I'm not sure we should immediately inline. Some level of simplification should occur on the individual functions first. Otherwise the code size is growing and increasing compile time everywhere. AlwaysInline doesn't immediately inline the function though. When after is the inliner actually run?


Right, we do not immediately inline, we just immediately mark all functions for inlining. The inliner pass runs in its regular place in opt. Also module passes work after function passes, so function simplifications happen.


Repository:
  rL LLVM

https://reviews.llvm.org/D31016





More information about the llvm-commits mailing list