[PATCH] D139644: [InlineAdvisor] Allow loading advisors as plugins

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 23 06:31:41 PST 2022


thakis added a comment.

In D139644#4003692 <https://reviews.llvm.org/D139644#4003692>, @thakis wrote:

>> Adds the ability to load InlineAdvisors as plugins. This allows developing and distributing inlining heuristics outside of tree.
>
> By the way, was there an RFC somewhere that discussed why this is a good thing for the project? (I have no opinion on this myself, but it clearly comes with a cost, so it'd be good if people working in this domain overall agreed that this is a good direction.)

BTW, was this ever answered?


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

https://reviews.llvm.org/D139644



More information about the llvm-commits mailing list