[clang] [CIR] Make ASTContext optional in runCIRToCIRPasses (PR #227128)

Konstantinos Parasyris via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 28 18:10:27 PDT 2026


================
@@ -111,15 +122,14 @@ runCIRToCIRPasses(mlir::ModuleOp theModule, mlir::MLIRContext &mlirContext,
   // outlines dynamic global initializers into functions.  It must run before
   // CallConvLowering so the classifier sees them, otherwise their signatures
   // go unclassified and caller and callee disagree on the ABI.
-  pm.addPass(mlir::createLoweringPreparePass(&astContext));
+  pm.addPass(mlir::createLoweringPreparePass(astContext));
----------------
koparasy wrote:

We need to figure this out, so keep commenting until we address these parts.

We do not need astcontext (today). The series of PRs cleaned lowering prepare from any AST-context requirement. My preference is to **not** have it in lowering prepare. @bcardosolopes please chime in.

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


More information about the cfe-commits mailing list