================
@@ -5130,22 +5130,25 @@ std::unique_ptr<mlir::Pass> 
createConvertCIRToLLVMPass() {
   return std::make_unique<ConvertCIRToLLVMPass>();
 }
 
-void populateCIRToLLVMPasses(mlir::OpPassManager &pm) {
+void populateCIRToLLVMPasses(mlir::OpPassManager &pm, bool enableOpenMP) {
   mlir::populateCIRPreLoweringPasses(pm);
-  pm.addPass(mlir::omp::createMarkDeclareTargetPass());
+  if (enableOpenMP)
+    pm.addPass(mlir::omp::createMarkDeclareTargetPass());
----------------
skatrak wrote:

Not a change request for this patch, only a related comment: I see there's no 
function filtering between this and the host op filtering pass. Is the plan to 
handle that in CIR or in LLVM dialect? The question here is mostly to figure 
out whether we might want to move that pass to the omp dialect as well.

https://github.com/llvm/llvm-project/pull/209592
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to