[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