[clang] [flang] [llvm] [mlir] Make MMIWP not have ownership over MMI + Make MMI Only Use an External MCContext (PR #105541)

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 20 05:19:24 PDT 2024


arsenm wrote:

> @aeubanks It's not impossible to separate them completely. `MCContext` is needed during initialization and finalization of the `MachineModuleInfoWrapperPass` (and its new pass manager variant) to set the diagnostics handler.
> 
> In theory, you can just pass the context to the wrapper pass instead. @arsenm any thoughts on this?

The MachineModuleInfo is the container for all the MachineFunctions (which do hold a reference to the MCContext), so it kind of makes sense to keep it there. But it does look like it should be simple to remove the reference here. So I would say it's better to just remove it

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


More information about the cfe-commits mailing list