================
@@ -319,6 +320,10 @@ class CodeGenRewrite : public 
fir::impl::CodeGenRewriteBase<CodeGenRewrite> {
       runOn(func, func.getBody());
     for (auto global : mod.getOps<fir::GlobalOp>())
       runOn(global, global.getRegion());
+    for (auto omp : mod.getOps<mlir::omp::ReductionDeclareOp>()) {
----------------
clementval wrote:

Wouldn't applying the patterns on the module directly work for all operation in 
it? So we could get rid of call to runOn. 

https://github.com/llvm/llvm-project/pull/84954
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
  • [... Tom Eccles via llvm-branch-commits
    • ... via llvm-branch-commits
    • ... Tom Eccles via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits
    • ... Tom Eccles via llvm-branch-commits
    • ... Tom Eccles via llvm-branch-commits
    • ... Kiran Chandramohan via llvm-branch-commits
    • ... Kiran Chandramohan via llvm-branch-commits
    • ... Tom Eccles via llvm-branch-commits
    • ... via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits
    • ... Valentin Clement バレンタイン クレメン via llvm-branch-commits

Reply via email to