[PATCH] D52716: [Inliner] Penalise inlining of calls with loops at Oz

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 2 14:10:36 PDT 2018


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

>From the discussion here and on https://reviews.llvm.org/D53405, it seems like the pieces we aren't modeling are hard to model well, and it will be hard to get them all right.  And I think you're right that it's unlikely to be profitable to inline loops.  So LGTM


https://reviews.llvm.org/D52716





More information about the llvm-commits mailing list