================
@@ -314,11 +314,8 @@ class CodeGenRewrite : public 
fir::impl::CodeGenRewriteBase<CodeGenRewrite> {
 
   void runOnOperation() override final {
     // Call runOn on all top level regions that may contain 
emboxOp/arrayCoorOp.
-    auto mod = getOperation();
-    for (auto func : mod.getOps<mlir::func::FuncOp>())
-      runOn(func, func.getBody());
-    for (auto global : mod.getOps<fir::GlobalOp>())
-      runOn(global, global.getRegion());
+    mlir::ModuleOp mod = getOperation();
+    runOn(mod);
----------------
clementval wrote:

nit: merge `ronOn()` in `runOnOperation()` since it has only a single use now. 

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
  • [... 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
  • [... Tom Eccles via llvm-branch-commits

Reply via email to