[PATCH] D23299: [PM] Port the always inliner to the new pass manager in a much more minimal and boring form than the old pass manager's version.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 11 11:37:34 PDT 2016


chandlerc added a comment.

In https://reviews.llvm.org/D23299#512749, @davidxl wrote:

> Why is this pass implemented as a module pass (instead of cgscc pass as before)? Is there any compile time concerns (at O0) ?


As I tried to explain in the patch description (and let me know if I should improve it): because a module pass is simpler, and avoids computing the call graph when we don't need it.

It might help compile time, but that wasn't my primary concern.


https://reviews.llvm.org/D23299





More information about the llvm-commits mailing list