[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