[llvm] [CodeGen] Make MCContext in MachineModuleInfo a unique_ptr. (PR #104834)

weiwei chen via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 19 13:06:19 PDT 2024


================
@@ -124,10 +124,10 @@ class MachineModuleInfo {
   const LLVMTargetMachine &getTarget() const { return TM; }
 
   const MCContext &getContext() const {
-    return ExternalContext ? *ExternalContext : Context;
+    return ExternalContext ? *ExternalContext : *Context;
----------------
weiweichen wrote:

Maybe? I don't understand why it's here either (and my PR is doing anything with it 😛). Any chance we can decide the fate of `ExternalContext` in a separate PR since it's out of the scope of this one? 

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


More information about the llvm-commits mailing list