[PATCH] D73994: [InlineCost] Relax bonus restrictions on uninlinable functions

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 13 11:38:03 PST 2020


davidxl added a comment.

I have concerns on so much plumbing changes to make the small heuristic change.

On the other hand, can you first extract the pass manager builder changes (i.e, passing Phase kind) into a separate patch?  That one seems independent.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73994/new/

https://reviews.llvm.org/D73994





More information about the llvm-commits mailing list