================
@@ -44,7 +44,7 @@ class CIRGenerator : public clang::ASTConsumer {
const clang::CodeGenOptions &codeGenOpts;
protected:
- std::unique_ptr<mlir::MLIRContext> mlirContext;
+ std::shared_ptr<mlir::MLIRContext> mlirContext;
----------------
andykaylor wrote:
Perhaps it's also relevant to note that I don't seem to need the context here
in the current patch. It's only used as a unique pointer in the path where
we're lowering CIR to other MLIR dialects before generating LLVM IR. So, I
could remove this part of the change and defer the decision until it is needed.
https://github.com/llvm/llvm-project/pull/124650
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
