[llvm] [NewPM][CodeGen] Let `ModuleToMachineFunctionPassAdaptor` own `MachineModuleInfo` (PR #87226)

via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 9 00:42:48 PDT 2024


paperchalice wrote:

> > `shared_ptr` seems better here.
> 
> Why? I generally think shared_ptr is not the answer

There might be several `<IR>ToMachineFunctionPassAdaptors` in pipeline, there is no benefit in considering ownership among these adapters, just let them share the `MachineModuleInfo`.

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


More information about the llvm-commits mailing list