[llvm] [Analysis] Remove global state from `PluginInline{Advisor,Order}Analysis`. (PR #114615)

Michele Scandale via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 18 15:38:55 PST 2024


michele-scandale wrote:

@aeubanks Thanks for merging the PR. I would have done that myself today.

> I'd also request that the comments [here](https://github.com/llvm/llvm-project/blob/5005f8d2486d6eec7b2b8ae04f49e8a87ebf4bf6/llvm/include/llvm/Analysis/InlineAdvisor.h#L241) be updated to use PassBuilder::registerAnalysisRegistrationCallback instead of adding a pass that registers the analysis (in a separate PR)

I'll prepare a separate PR for this part.

https://github.com/llvm/llvm-project/pull/114615


More information about the llvm-commits mailing list