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

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 4 11:30:56 PST 2024


aeubanks wrote:

I agree that global state breaks a bunch of cases that we care about, but using the pass manager infrastructure to decide whether or not to use the default vs ML inline advisor seems like a hack to avoid having to create the proper APIs. IMO each inliner instance should decide if it's going to use the ML inliner or not, not each pipeline. Thoughts?

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


More information about the llvm-commits mailing list